[
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)
