rdblue commented on code in PR #12298:
URL: https://github.com/apache/iceberg/pull/12298#discussion_r2258317951


##########
parquet/src/main/java/org/apache/iceberg/parquet/Parquet.java:
##########
@@ -155,42 +171,28 @@ public static WriteBuilder write(EncryptedOutputFile 
file) {
     }
   }
 
+  /**
+   * @deprecated Since 1.10.0, will be removed in 1.11.0. Use {@link
+   *     FormatModelRegistry#writeBuilder(FileFormat, String, 
EncryptedOutputFile)} instead.
+   */
+  @Deprecated

Review Comment:
   I'm not sure that this is much of a problem if we have new shared interfaces 
for producing data and delete writers that just wrap this. I think we should 
probably revisit this when that structure is finalized.



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