zhjwpku opened a new pull request, #763: URL: https://github.com/apache/iceberg-cpp/pull/763
While reviewing #614, I noticed that `PlanErrorHandler::Accept` does not match the behavior of the Java implementation. A detailed comparison with Java's `ErrorHandlers` revealed several gaps between iceberg-cpp and Iceberg Java. Some of these are oversights in the original implementation, while others correspond to improvements that were made later in Iceberg Java, including: * https://github.com/apache/iceberg/pull/13143 * https://github.com/apache/iceberg/pull/14927 * https://github.com/apache/iceberg/pull/15051 * https://github.com/apache/iceberg/pull/16059 This PR closes those gaps and brings the error handling behavior in iceberg-cpp closer to the Java implementation. -- 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]
