Skip to content

ARROW-11959: [Rust][DataFusion] Fix log line#9699

Closed
Dandandan wants to merge 1 commit intoapache:masterfrom
Dandandan:fix_logging
Closed

ARROW-11959: [Rust][DataFusion] Fix log line#9699
Dandandan wants to merge 1 commit intoapache:masterfrom
Dandandan:fix_logging

Conversation

@Dandandan
Copy link
Copy Markdown
Contributor

Just a small fix to fix a debug line. It was printing the unoptimized plan instead of the optimized plan.

@github-actions
Copy link
Copy Markdown

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #9699 (eb78919) into master (6a13206) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9699   +/-   ##
=======================================
  Coverage   82.57%   82.57%           
=======================================
  Files         248      248           
  Lines       58279    58279           
=======================================
+ Hits        48121    48122    +1     
+ Misses      10158    10157    -1     
Impacted Files Coverage Δ
rust/datafusion/src/execution/context.rs 90.84% <100.00%> (ø)
rust/parquet/src/encodings/encoding.rs 95.05% <0.00%> (+0.19%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a55ef2b...eb78919. Read the comment docs.

Copy link
Copy Markdown
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Dandandan !

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.57%. Comparing base (6a13206) to head (eb78919).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9699   +/-   ##
=======================================
  Coverage   82.57%   82.57%           
=======================================
  Files         248      248           
  Lines       58279    58279           
=======================================
+ Hits        48121    48122    +1     
+ Misses      10158    10157    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants