[GitHub] [drill] dzamo commented on a change in pull request #2351: DRILL1282: Move parquet to use v2 format as default

[GitBox <gi...@apache.org>]

dzamo commented on a change in pull request #2351:
URL: https://github.com/apache/drill/pull/2351#discussion_r738475956



##########
File path: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java
##########
@@ -375,6 +375,12 @@ private ExecConstants() {
   public static final OptionValidator PARQUET_WRITER_LOGICAL_TYPE_FOR_DECIMALS_VALIDATOR = new EnumeratedStringValidator(PARQUET_WRITER_LOGICAL_TYPE_FOR_DECIMALS,
       new OptionDescription("Parquet writer logical type for decimal; supported types \'fixed_len_byte_array\' and \'binary\'"),
       "fixed_len_byte_array", "binary");
+  public static final String PARQUET_WRITER_FORMAT_VERSION = "store.parquet.writer.format_version";
+  public static final OptionValidator PARQUET_WRITER_FORMAT_VERSION_VALIDATOR = new EnumeratedStringValidator(
+    PARQUET_WRITER_FORMAT_VERSION,
+    new OptionDescription("Parquet format version used for storing Parquet output.  Allowed values: PARQUET_1_0, PARQUET_2_0"),
+    "PARQUET_1_0", "PARQUET_2_0"

Review comment:
       @vdiravka nice, thanks!  I'll do this...
   




-- 
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: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org