pitrou commented on code in PR #35886:
URL: https://github.com/apache/arrow/pull/35886#discussion_r1254509177
##########
cpp/src/parquet/column_writer.h:
##########
@@ -85,6 +87,22 @@ class PARQUET_EXPORT PageWriter {
public:
virtual ~PageWriter() {}
+ static std::unique_ptr<PageWriter> Open(
+ std::shared_ptr<ArrowOutputStream> sink, Compression::type codec,
+ ColumnChunkMetaDataBuilder* metadata, int16_t row_group_ordinal = -1,
+ int16_t column_chunk_ordinal = -1,
+ ::arrow::MemoryPool* pool = ::arrow::default_memory_pool(),
+ bool buffered_row_group = false,
+ std::shared_ptr<Encryptor> header_encryptor = NULLPTR,
+ std::shared_ptr<Encryptor> data_encryptor = NULLPTR,
+ bool page_write_checksum_enabled = false,
+ // column_index_builder MUST outlive the PageWriter
+ ColumnIndexBuilder* column_index_builder = NULLPTR,
+ // offset_index_builder MUST outlive the PageWriter
+ OffsetIndexBuilder* offset_index_builder = NULLPTR,
+ const CodecOptions& codec_options = CodecOptions{});
+
+ ARROW_DEPRECATED("Use the one with CodecOptions instead")
Review Comment:
```suggestion
ARROW_DEPRECATED("Deprecated in 13.0.0. Use CodecOptions-taking overload
instead.")
```
--
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]
