dongkelun commented on code in PR #7339:
URL: https://github.com/apache/hudi/pull/7339#discussion_r1035773214
##########
hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/DataSourceOptions.scala:
##########
@@ -62,6 +62,11 @@ object DataSourceReadOptions {
"(or) Read Optimized mode (obtain latest view, based on base files) (or)
Snapshot mode " +
"(obtain latest view, by merging base and (if any) log files)")
+ val QUERY_USE_DATABASE: ConfigProperty[Boolean] = ConfigProperty
+ .key("hoodie.query.use.database")
+ .defaultValue(false)
+ .withDocumentation("Whether to add database name to qualify table name
when setting parameters in Spark SQL query")
+
Review Comment:
@xiarixiaoyao This title is not reflected because the form of set parameter
is not supported previously. Adding this parameter is mainly consistent with
Hive incremental query: ` HoodieHiveUtils.HOODIE_ INCREMENTAL_ USE_ DATABASE ',
mainly considering the case that different databases have the same table name.
The reason why it is not described in detail in the PR is that it is
uncertain whether the community will approve this form of query. If necessary,
I can add a detailed description in the PR. In addition, only incremental
queries are added to the test cases, excluding other query types. If necessary,
I can add more detailed test cases
--
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]
