[ https://issues.apache.org/jira/browse/CALCITE-4821?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Vladimir Sitnikov resolved CALCITE-4821. ---------------------------------------- Fix Version/s: 1.28.0 Resolution: Fixed Fixed in https://github.com/apache/calcite/pull/2558/commits/ef40b524936d9d0497deebcb5be2139502a052e1 > Move utility test classes into calcite-testkit and unpublish -test.jar > ---------------------------------------------------------------------- > > Key: CALCITE-4821 > URL: https://issues.apache.org/jira/browse/CALCITE-4821 > Project: Calcite > Issue Type: Improvement > Components: core > Reporter: Vladimir Sitnikov > Priority: Major > Labels: pull-request-available > Fix For: 1.28.0 > > Time Spent: 20m > Remaining Estimate: 0h > > -test.jar make it hard to support build configuration, and it blocks > migration to Gradle 7. > As we move the commonly used classes to its own -testkit module, it would > make it easier to reuse test helpers, it would make easier to recognize the > Test API vs {{@Test}} code, and it would make certain classes smaller. E.g. > CatchAllSchema is currently located in ReflectiveSchemaTest, and if we move > it to a top-level class, then ReflectiveSchemaTest would be smaller. -- This message was sent by Atlassian Jira (v8.3.4#803005)