clintropolis commented on a change in pull request #11812:
URL: https://github.com/apache/druid/pull/11812#discussion_r732461462



##########
File path: distribution/pom.xml
##########
@@ -417,6 +417,8 @@
                                         <argument>-c</argument>
                                         
<argument>org.apache.druid.extensions.contrib:statsd-emitter</argument>
                                         <argument>-c</argument>
+                                        
<argument>org.apache.druid.extensions.contrib:prometheus-emitter</argument>

Review comment:
       good catch.
   
   I did a quick check and it looks like we are also missing 
`druid-momentsketch`, would you mind adding it too? 
   
   `druid-testing-tools` is also not in any profile, but I'm not sure it 
actually makes sense to include anyway so we should ignore it for now, because 
its primary purpose is to aid in writing integration tests since it includes 
mechanisms to control query execution time.




-- 
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]

Reply via email to