Skip to content

chore(integrations/parquet): add README#4980

Merged
Xuanwo merged 3 commits intoapache:mainfrom
WenyXu:chore/integrations-parquet-readme
Aug 8, 2024
Merged

chore(integrations/parquet): add README#4980
Xuanwo merged 3 commits intoapache:mainfrom
WenyXu:chore/integrations-parquet-readme

Conversation

@WenyXu
Copy link
Copy Markdown
Member

@WenyXu WenyXu commented Aug 7, 2024

Which issue does this PR close?

close #4868

Rationale for this change

What changes are included in this PR?

Add README for integrations/parquet

Are there any user-facing changes?

No

@WenyXu WenyXu requested review from PsiACE and Xuanwo as code owners August 7, 2024 17:34
@github-actions github-actions Bot added the releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell label Aug 7, 2024
Copy link
Copy Markdown
Member

@PsiACE PsiACE left a comment

Choose a reason for hiding this comment

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

LGTM

@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Aug 8, 2024

Would you like to also add parquet_opendal in the For ANY integrations section of our README?

Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Perfect, thanks! Also thanks for @PsiACE's review.

@Xuanwo Xuanwo merged commit ccfef2b into apache:main Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell

Projects

None yet

Development

Successfully merging this pull request may close these issues.

idea: Add parquet arrow async write/read support

3 participants