[GitHub] [iceberg] vvellanki commented on issue #1832: Finalise the proposal for including Partition index for Iceberg tables

[GitBox <gi...@apache.org>]

vvellanki commented on issue #1832:
URL: https://github.com/apache/iceberg/issues/1832#issuecomment-740492401


   @wangmiao1981 I was referring to the [following](https://docs.google.com/document/d/1G6GeOXkGSiSTcu0lDS6VA1FtJ_uz9FO4tF2Pffmx9LU/edit#heading=h.48c09kgpva06) when I refer to Partition Index.
   
   Reading your document, these two appear to be quite different:
   - The partition index proposal is at the table scope; while secondary indexes are per-file, per-column scope
   - The secondary indexes themselves are files - one or more files per data file; as opposed to the partition index (one per snapshot)
   
   @rdblue @wangmiao1981 I see the Partition Index and Secondary Indexes as being sufficiently different that we should treat them independently


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org