wgtmac commented on code in PR #1339:
URL: https://github.com/apache/parquet-mr/pull/1339#discussion_r1587517676


##########
parquet-hadoop/src/main/java/org/apache/parquet/hadoop/ParquetWriter.java:
##########
@@ -503,8 +503,7 @@ protected Builder(OutputFile path) {
      * @return an appropriate WriteSupport for the object model.
      */
     protected WriteSupport<T> getWriteSupport(ParquetConfiguration conf) {
-      throw new UnsupportedOperationException(

Review Comment:
   Is it possible to use a flag from `ParquetConfiguration` to determine if 
backward compatibility is required? By default we should enable backward 
compatibility and disable it in the parquet-mr 2.0.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to