github-actions[bot] commented on code in PR #63747: URL: https://github.com/apache/doris/pull/63747#discussion_r3338369540
########## regression-test/conf/regression-conf.groovy: ########## @@ -332,6 +332,11 @@ hudiHmsPort=19083 hudiMinioPort=19100 hudiMinioAccessKey="minio" hudiMinioSecretKey="minio123" +otherConfigs = [ + enableKafkaTest: "true", Review Comment: This `otherConfigs` map is not copied into `context.config.otherConfigs` by the regression framework. `Config.fromConfigObject()` excludes declared fields such as `otherConfigs` when it copies script properties into `config.otherConfigs`, and it never merges `obj.otherConfigs` separately, so the existing top-level `enableKafkaTest=false` / `kafka_port=19193` remain the values seen by suites. If this PR needs the new routine-load case enabled by this config, set the top-level properties instead; otherwise remove this dead block to avoid misleading local runs. -- 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]
