HeartLinked commented on PR #238: URL: https://github.com/apache/iceberg-cpp/pull/238#issuecomment-3317207140
Thanks for your contribution! I noticed part of the code logic in your PR overlaps with mine (https://github.com/apache/iceberg-cpp/pull/185). My PR has been in review for a long time, and it’s now almost ready for merge. It shouldn’t take long to finalize. To avoid unnecessary code conflicts and make our collaboration more efficient, could you wait until my PR is merged? Then you can rebase your code on the merged version and only focus on adding the new Literal support for decimal. This way, your core contribution is preserved, and we reduce maintenance costs from duplicate code. Thanks! -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
