CALCITE-6465: Rework code generator to use Flink code splitter
This builds but introduces a dependency on flink-core. Looking for a way around this.
Quality Gate passed
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
83.0% Coverage on New Code
0.0% Duplication on New Code
@rubenada @julianhyde , just wondering if there was some more feedback on this and would like some clarification on where system tests reside within Calcite. Thanks.
@jduo , not sure what you mean by "system tests". If you want to add a test with a large dynamic code that fails when this feature is turn off / passes when it is turned on; maybe you could check and adapt the test that was done for the slightly related CALCITE-3094 (and then refined in CALCITE-6593 ): LargeGeneratedJoinTest. The query in that test, with the JOIN_SELECTOR_COMPACT_CODE_THRESHOLD disabled, should throw the "code grows beyond 64 KB InternalCompilerException". At that point, turning on the code splitter from this PR, should make the test pass.
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 90 days if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.