Prajwal-banakar commented on PR #3319: URL: https://github.com/apache/fluss/pull/3319#issuecomment-4540790356
Hi @polyzos Thank you for the improvements! the ITCase is a much cleaner solution for validating the planner contract, and the serialize fix is cleaner too. LGTM! the failing CI check is in fluss-server (JVM crash: "pure virtual method called" from native RocksDB code), which is unrelated to this PR's changes in fluss-flink-common. This looks like a pre-existing runner flake -- 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]
