yihua commented on PR #13414:
URL: https://github.com/apache/hudi/pull/13414#issuecomment-2978164730

   > @yihua @Davis-Zhang-Onehouse
   > 
   > I prefer not to introduce a new concept like `partitionStrategy`, that may 
or may not be changing/defined per index.
   > 
   > Instead I propose
   > 
   > * We introduce a `layoutVersion` field into the index definition. If the 
field is not present, then value `version 1` is assumed.
   > * We handled the differences between SI layouts `1` and `2`, inside the 
reader/write code. Inside the code, the layout version is handled specifically 
for each index type.
   > * We document different index storage layouts in the 1.0-tech-specs (needs 
a docs PR to asf-site branch)
   > * We still need to bump up the table version (to handle a downgrade where 
SI layout 2, can't be read by releases < 1.1)
   
   Actually this proposed index layout version is more general.  I prefer this 
too.


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to