pitrou commented on code in PR #36955:
URL: https://github.com/apache/arrow/pull/36955#discussion_r1287373283
##########
cpp/src/parquet/properties.h:
##########
@@ -135,14 +136,13 @@ static constexpr int64_t DEFAULT_WRITE_BATCH_SIZE = 1024;
static constexpr int64_t DEFAULT_MAX_ROW_GROUP_LENGTH = 1024 * 1024;
static constexpr bool DEFAULT_ARE_STATISTICS_ENABLED = true;
static constexpr int64_t DEFAULT_MAX_STATISTICS_SIZE = 4096;
-static constexpr Encoding::type DEFAULT_ENCODING = Encoding::PLAIN;
static const char DEFAULT_CREATED_BY[] = CREATED_BY_VERSION;
static constexpr Compression::type DEFAULT_COMPRESSION_TYPE =
Compression::UNCOMPRESSED;
static constexpr bool DEFAULT_IS_PAGE_INDEX_ENABLED = false;
class PARQUET_EXPORT ColumnProperties {
public:
- ColumnProperties(Encoding::type encoding = DEFAULT_ENCODING,
+ ColumnProperties(std::optional<Encoding::type> encoding = std::nullopt,
Review Comment:
Hmm... if we want to switch to `std::optional` in `ColumnProperties` we
should probably do so more consistently, instead of breaking compatibility for
this single property. Can this be deferred to another issue and PR?
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
