Uh oh!
There was an error while loading. Please reload this page.
PARQUET-204: add parquet-schema directory support - #136
Conversation
2831d2a to
361cf63Comparetsdeng
commented
Mar 6, 2015
This is good. +1 |
There was a problem hiding this comment.
As this is stateless there could be one static instance of it.
HIDDEN_FILE_FILTER = new HiddenFileFilter();
julienledem
commented
Mar 6, 2015
Thanks for cleaning up all this duplicated code. |
361cf63 to
633829bComparenevillelyh
commented
Mar 9, 2015
I changed it to a static INSTANCE member. |
There was a problem hiding this comment.
Should we take schema evolution into account here? I.e., show the merged schemas of all part-files.
There was a problem hiding this comment.
I don't think so. There's no guarantee that there is a single schema for the data and merging all of the schemas into one would be misleading: a union strategy can produce a schema that can't be satisfied (as a column projection) by any of the files. I think it's best to return one or all of the unique schemas, but this is already going slightly beyond what Parquet itself should be doing as a file format. Parquet reads and writes files, while Hive or Kite manages the data as a collection.
rdblue
commented
Mar 24, 2015
Thanks @nevillelyh! |
No description provided.