You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@parquet.apache.org by GitBox <gi...@apache.org> on 2021/04/01 09:12:07 UTC

[GitHub] [parquet-mr] gszadovszky commented on a change in pull request #886: PARQUET-2012 Mark ProtoParquetWriter constructors deprecated

gszadovszky commented on a change in pull request #886:
URL: https://github.com/apache/parquet-mr/pull/886#discussion_r605496828



##########
File path: parquet-protobuf/src/main/java/org/apache/parquet/proto/ProtoParquetWriter.java
##########
@@ -43,6 +43,8 @@
    * @param blockSize            HDFS block size
    * @param pageSize             See parquet write up. Blocks are subdivided into pages for alignment and other purposes.
    * @throws IOException if there is an error while writing
+   *
+   * @deprecated will be removed in 2.0.0.; Used ProtoParquetWriter.Builder instead

Review comment:
       Please, also use the java annotation `@Deprecated` for all the deprecated methods.




-- 
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:
users@infra.apache.org