maytasm commented on a change in pull request #10171:
URL: https://github.com/apache/druid/pull/10171#discussion_r459059440



##########
File path: integration-tests/docker/druid.sh
##########
@@ -82,14 +83,18 @@ setupData()
   # The "query" and "security" test groups require data to be setup before 
running the tests.
   # In particular, they requires segments to be download from a pre-existing 
s3 bucket.
   # This is done by using the loadSpec put into metadatastore and s3 
credientials set below.
-  if [ "$DRUID_INTEGRATION_TEST_GROUP" = "query" ] || [ 
"$DRUID_INTEGRATION_TEST_GROUP" = "security" ]; then
+  if [ "$DRUID_INTEGRATION_TEST_GROUP" = "query" ] || [ 
"$DRUID_INTEGRATION_TEST_GROUP" = "query-retry" ] || [ 
"$DRUID_INTEGRATION_TEST_GROUP" = "security" ]; then

Review comment:
       I am imaging the flags as easy control to the Druid cluster. Basically, 
there's a list of all the flags on the README and if you are writing a new test 
group, you can easily pick and choose how you want the state of your Druid 
cluster to be. i.e. you need to start x service then set this flag, you need 
pre-load data set this flag, you need hadoop set this flag, you want indexer 
instead of MM set this flag, etc.




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

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