You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/09/30 16:02:32 UTC

[GitHub] [druid] churromorales commented on a diff in pull request #13156: Support for middle manager less druid, tasks launch as k8s jobs

churromorales commented on code in PR #13156:
URL: https://github.com/apache/druid/pull/13156#discussion_r984740714


##########
distribution/docker/peon.sh:
##########
@@ -0,0 +1,153 @@
+#!/bin/sh

Review Comment:
   So when creating this patch, I was hoping to minimize core changes to increase the odds of this patch getting accepted.  In all honesty, I believe if this is a first class feature, we can fix some other hacky stuff that was done.  I definitely think having a Cli command for the task is a good idea, I think this is something that I could put up a PR if /when this gets merged.  



-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org