lxy-9602 opened a new pull request, #263:
URL: https://github.com/apache/paimon-cpp/pull/263
<!-- PR titles must follow Conventional Commits: <type>(<optional-scope>):
<description> -->
### Purpose
<!-- Linking this pull request to the issue -->
This change aligns several format-related behaviors with the latest Java
Paimon implementation:
- Keep legacy manifest formats readable, but allow only Avro for manifest
writes.
- `manifest.format` is retained as a deprecated compatibility option.
- Writing with a non-Avro manifest format now returns an explicit error.
- Read paths continue to use the configured legacy format.
- Convert the `parquet/append_09` test fixture manifests from ORC to Avro
while preserving the existing manifest filenames and snapshots.
- Rename the Parquet dictionary option from `parquet.enable-dictionary` to
`parquet.enable.dictionary`, matching Java Paimon.
- Align `listagg` blank-string handling with Java:
- Add `StringUtils::IsBlank` with Java `Character.isWhitespace` semantics.
- Ignore empty, whitespace-only, and supported Unicode blank inputs.
- Prevent blank accumulators from producing a leading delimiter.
- Ignore blank tokens when `distinct` is enabled.
<!-- What is the purpose of the change -->
### Tests
Added or updated unit tests covering:
- read and write access for Avro and legacy manifest formats
- rejection of non-Avro manifest writes
- disabling Parquet dictionary encoding with `parquet.enable.dictionary`
- ASCII and Unicode behavior of `StringUtils::IsBlank`
- `listagg` handling of blank inputs, accumulators, and distinct tokens
- commit tests using the Avro-converted `parquet/test_data` fixture
<!-- List UT and IT cases to verify this change -->
### API and Format
Behavioral and format-related changes:
- Avro is now the only writable manifest format.
- Existing non-Avro manifests remain readable.
- The Parquet dictionary option is renamed to `parquet.enable.dictionary`.
<!-- Does this change affect API in include dir or storage format or
protocol -->
### Documentation
<!-- Does this change introduce a new feature -->
### Generative AI tooling
Generated-by: Codex (GPT-5)
<!--
If generative AI tooling has been used in the process of authoring this
patch, please include the
phrase: 'Generated-by: ' followed by the name of the tool and its version.
If no, write 'No'.
Please refer to the [ASF Generative Tooling
Guidance](https://www.apache.org/legal/generative-tooling.html) for details.
-->
--
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]
