leerho opened a new issue #6866: Reverse dependencies from /main/ to /test/ URL: https://github.com/apache/incubator-druid/issues/6866 Discussed in #6814 Druid-benchmarks has a number of reverse dependencies from the main tree back to the test-tree. This is bad practice and should be corrected. An even larger question is why is Druid-benchmarks even in the main tree? Specifically: ``` druid-benchmarks org.apache.druid.benchmark ExpressionAggregationBenchmark import org.apache.druid.query.expression.TestExprMacroTable; //located in test tree ExpressionSelectorBenchmark import org.apache.druid.query.expression.TestExprMacroTable; //located in test tree org.apache.druid.benchmark.datagen SegmentGenerator import org.apache.druid.segment.IndexBuilder; //located in test tree import org.apache.druid.segment.TestHelper; //located in test tree //located in druid-processing-0.13.0-incubating-SNAPSHOT-tests.jar org.apache.druid.benchmark.query SqlBenchmark import org.apache.druid.sql.calcite.util.CalciteTests; //located in test tree import org.apache.druid.sql.calcite.util.SpecificSegmentsQuerySegmentWalker; //located in test tree //located in druid-sql-0.13.0-incubating-SNAPSHOT-tests.jar ```
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
