slfan1989 opened a new issue, #14: URL: https://github.com/apache/paimon-cpp/issues/14
### Search before asking - [x] I searched in the [issues](https://github.com/apache/paimon-cpp/issues) and found nothing similar. ### Paimon-cpp version Java Paimon's `RowKind.fromShortString` normalizes the input with `toUpperCase()`, so both uppercase and lowercase short strings are accepted. For example, in Java: ``` RowKind.fromShortString("+i") RowKind.fromShortString("-u") RowKind.fromShortString("+u") RowKind.fromShortString("-d") ``` can be parsed successfully. However, in paimon-cpp, RowKind::FromShortString currently only accepts exact uppercase strings: ``` RowKind::FromShortString("+I") RowKind::FromShortString("-U") RowKind::FromShortString("+U") RowKind::FromShortString("-D") ``` Lowercase inputs such as "+i" or "-d" return an invalid status. ### Minimal reproduce step In paimon-cpp, RowKind::FromShortString only matches exact uppercase strings. The following calls return an invalid status: ``` auto insert = RowKind::FromShortString("+i"); auto update_before = RowKind::FromShortString("-u"); auto update_after = RowKind::FromShortString("+u"); auto delete_kind = RowKind::FromShortString("-d"); ``` Expected: these lowercase short strings should be parsed successfully, consistent with Java Paimon. Actual: they are rejected as unsupported short strings. ### What doesn't meet your expectations? Expected behavior: - "+i" should map to INSERT - "-u" should map to UPDATE_BEFORE - "+u" should map to UPDATE_AFTER - "-d" should map to DELETE Actual behavior: - "+i", "-u", "+u", and "-d" return an invalid status. ### Anything else? _No response_ ### Are you willing to submit a PR? - [ ] I'm willing to submit a 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
