vvysotskyi commented on a change in pull request #1985: DRILL-7565: ANALYZE
TABLE ... REFRESH METADATA does not work for empty Parquet files
URL: https://github.com/apache/drill/pull/1985#discussion_r380077493
##########
File path:
exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java
##########
@@ -511,6 +511,11 @@ private ExecConstants() {
new OptionDescription("Available as of Drill 1.17. Sets the implicit
column name for the lastModifiedTime column. " +
"For internal usage when producing Metastore analyze."));
+ public static final String IMPLICIT_PROJECT_METADATA_COLUMN_LABEL =
"drill.exec.storage.implicit.project_metadata.column.label";
+ public static final OptionValidator
IMPLICIT_PROJECT_METADATA_COLUMN_LABEL_VALIDATOR = new
StringValidator(IMPLICIT_PROJECT_METADATA_COLUMN_LABEL,
+ new OptionDescription("Available as of Drill 1.18. Sets the implicit
column name for the $project_metadata$ column. " +
Review comment:
Good point about that. I specified version here and in other places to be
consistent with other options descriptions.
I think adding version in options descriptions was done to simplify updating
docs for Drill Web site - there is no need to look up for the commit date and
version of Drill, where it was added, just copy and paste it from Drill Web-UI,
or from this class.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
