lxy-9602 commented on PR #253: URL: https://github.com/apache/paimon-cpp/pull/253#issuecomment-5447902615
The current changes seem reasonable to me. That said, to avoid `PkSortedBucketIndexState` in C++ gradually diverging too much from the Java implementation and becoming harder to align later — for example, if additional features or refactors are introduced on the Java side in the future — I’d suggest merging this logic into `paimon-java` first, and then bringing it into C++. -- 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]
