lxy-9602 opened a new pull request, #249:
URL: https://github.com/apache/paimon-cpp/pull/249

   <!-- PR titles must follow Conventional Commits: <type>(<optional-scope>): 
<description> -->
   
   ### Purpose
   
   <!-- Linking this pull request to the issue -->
   This PR consolidates duplicated string and option handling into the common 
utility classes.
   
   The main changes are:
   
   - Align `StringUtils` edge-case behavior with the corresponding Java 
implementation, including replace, trim, and starts-with operations.
   - Add ASCII-based case conversion and case-insensitive comparison helpers.
   - Add `OptionsUtils::GetNonEmptyValueFromMap` and improve prefixed option 
extraction.
   - Replace duplicated string and option handling across REST, filesystem, 
schema validation, variant, and system-table code.
   - Reuse `SpecialFields::CompleteSequenceAndValueKindField` when constructing 
the postpone-bucket write schema.
   - Stop injecting `parquet.read.enable-pre-buffer=false` into compaction read 
options. Parquet readers now respect the configured value or the default value.
   
   
   <!-- What is the purpose of the change -->
   
   ### Tests
   Added or updated unit coverage for:
   
   - `StringUtils` edge cases and case-insensitive comparison.
   - `OptionsUtils` non-empty values and prefixed option extraction.
   - Case-insensitive local filesystem schemes.
   - Case-insensitive Parquet format validation.
   - Sequence snapshot property parsing.
   <!-- List UT and IT cases to verify this change -->
   
   ### API and Format
   
   <!-- 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: OpenAI 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]

Reply via email to