PARQUET-2479: Update README with link to parquet website, clarify contents#243
Merged
Conversation
alamb
commented
May 19, 2024
| [Apache Thrift] definitions to read and write Parquet metadata. | ||
|
|
||
| Parquet metadata is encoded using Apache Thrift. | ||
| Apache Parquet is an open source, column-oriented data file format |
Contributor
Author
There was a problem hiding this comment.
This is the same wording from apache/parquet-site#59
14844e1 to
6079309
Compare
vinooganesh
approved these changes
May 19, 2024
Contributor
vinooganesh
left a comment
There was a problem hiding this comment.
+1, looks good to me
Contributor
Author
|
Thank you for the review @wgtmac and @vinooganesh |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://issues.apache.org/jira/browse/PARQUET-2479
As part of clarifying and improving the public communication about Parquet, I think we should also link parquet-format to the parquet.apache.org website
Also, the text in the format repo's readme is somewhat unclear on what is in the repo and how it related to other projects, so I clarified that as well
You can see the rendered version of the README here:
https://github.com/alamb/parquet-format/tree/alamb/doc_link_readme?tab=readme-ov-file#parquet-
Jira
Commits
Documentation
This is a README only change