slfan1989 commented on code in PR #2031:
URL: https://github.com/apache/auron/pull/2031#discussion_r2934850314
##########
spark-extension/src/main/java/org/apache/auron/spark/configuration/SparkAuronConfiguration.java:
##########
@@ -346,6 +346,12 @@ public class SparkAuronConfiguration extends
AuronConfiguration {
.withDescription("Enable PaimonScanExec operation conversion to
native Auron implementations.")
.withDefaultValue(true);
+ public static final ConfigOption<Boolean> ENABLE_HUDI_SCAN = new
SQLConfOption<>(Boolean.class)
+ .withKey("auron.enable.hudi.scan")
Review Comment:
Thanks for the comment! Currently, we only support `COW` tables, and `MOR`
is explicitly excluded in `HudiScanSupport`. So for this PR, a single
`auron.enable.hudi.scan` gate is sufficient. If `MOR` support is added in the
future, we can introduce separate toggles for different table types then.
--
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]
