see-quick commented on code in PR #22034:
URL: https://github.com/apache/kafka/pull/22034#discussion_r3303063072


##########
tests/docker/run_tests.sh:
##########
@@ -40,13 +39,6 @@ else
     export KAFKA_MODE="jvm"
 fi
 
-if [ "$REBUILD" == "t" ]; then
-    ./gradlew clean systemTestLibs
-    if [ "$KAFKA_MODE" == "native" ]; then
-        ./gradlew clean releaseTarGz
-    fi
-fi
-

Review Comment:
   Sure I can add also the `--clean-build` here and move logic of 
`systemTestLibs` and `releaseTarGz` to ducker-ak. 
   
   > run_tests.sh should simply convert the legacy REBUILD=t into the 
--clean-build flag when calling ducker-ak
   
   or simply just export CLEAN_BUILD i.e.,:
   ```bash
   if [ "$REBUILD" == "t" ]; then
       export CLEAN_BUILD=true
   fi
   ```



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

Reply via email to