Skip to content

docs: Add documentation for accelerating Iceberg Parquet scans with Comet [branch-0.8] #1696

Merged
andygrove merged 2 commits intoapache:branch-0.8from
andygrove:backport-1683
Apr 29, 2025
Merged

docs: Add documentation for accelerating Iceberg Parquet scans with Comet [branch-0.8] #1696
andygrove merged 2 commits intoapache:branch-0.8from
andygrove:backport-1683

Conversation

@andygrove
Copy link
Copy Markdown
Member

@andygrove andygrove commented Apr 29, 2025

Which issue does this PR close?

N/A

Rationale for this change

Backport #1683 to branch-0.8

What changes are included in this PR?

How are these changes tested?

@andygrove andygrove changed the title [branch-0.8] docs: Add documentation for accelerating Iceberg Parquet scans with Comet docs: Add documentation for accelerating Iceberg Parquet scans with Comet [branch-0.8] Apr 29, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 29, 2025

Codecov Report

❌ Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (branch-0.8@680e75a). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...ain/java/org/apache/comet/CometSchemaImporter.java 0.00% 2 Missing ⚠️
...rg/apache/arrow/c/AbstractCometSchemaImporter.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             branch-0.8    #1696   +/-   ##
=============================================
  Coverage              ?   58.82%           
  Complexity            ?     1090           
=============================================
  Files                 ?      126           
  Lines                 ?    12602           
  Branches              ?     2362           
=============================================
  Hits                  ?     7413           
  Misses                ?     4018           
  Partials              ?     1171           

☔ 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.

@andygrove andygrove requested a review from huaxingao April 29, 2025 19:32
Copy link
Copy Markdown
Contributor

@huaxingao huaxingao left a comment

Choose a reason for hiding this comment

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

LGTM Thanks @andygrove

Copy link
Copy Markdown
Contributor

@hsiang-c hsiang-c left a comment

Choose a reason for hiding this comment

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

LGTM


## Supported Storages

## Local
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

(nit) Should we keep ###Local?

@andygrove andygrove merged commit b6342b4 into apache:branch-0.8 Apr 29, 2025
78 checks passed
@andygrove andygrove deleted the backport-1683 branch July 10, 2025 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants