loquisgon commented on a change in pull request #11294:
URL: https://github.com/apache/druid/pull/11294#discussion_r662539405
##########
File path:
indexing-service/src/test/java/org/apache/druid/indexing/appenderator/BatchAppenderatorDriverTest.java
##########
@@ -17,7 +17,7 @@
* under the License.
*/
-package org.apache.druid.segment.realtime.appenderator;
+package org.apache.druid.indexing.appenderator;
Review comment:
I am excluding the class again.. I see no easy way to moving them all to
the same package. We are fighting against a tool (coverage) primarily,
secondarily against module design (i.e. "server" has some batch stuff and it
should only have "realtime" stuff). Resolving the latter, which takes
non-trivial effort and should go in another ticket, will also resolve the
former.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
