pengzhiwei2018 commented on a change in pull request #3330:
URL: https://github.com/apache/hudi/pull/3330#discussion_r675337941
##########
File path:
hudi-common/src/main/java/org/apache/hudi/common/model/WriteOperationType.java
##########
@@ -48,6 +48,8 @@
INSERT_OVERWRITE_TABLE("insert_overwrite_table"),
// compact
COMPACT("compact"),
+ // z-order
Review comment:
Do we need another write type for z-order? For my first think, `z-order`
is a layout optimize for the table which is much similar to `order-by`. As
currently we already have the `clustering` to optimize the layout by
`order-by`, can we implement the z-order by under the `clustering` ? So that we
can support two kind of data clustering:
> Clustering table order by col1;
> Clustering table zorder by col2;
--
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]
