XComp commented on code in PR #19528: URL: https://github.com/apache/flink/pull/19528#discussion_r859581561
########## tools/ci/stage.sh: ########## @@ -21,8 +21,8 @@ STAGE_COMPILE="compile" STAGE_CORE="core" STAGE_PYTHON="python" STAGE_TABLE="table" -STAGE_CONNECTORS="connectors" -STAGE_KAFKA_GELLY="kafka/gelly" +STAGE_CONNECTORS_1="connectors_1" +STAGE_CONNECTORS_2="connectors_2" Review Comment: I mean, in the end it depends on the screen and font size of the user. One could also argue that `test_ci` is already problematic as a prefix for the setup I had in mind. Considering that (and agreeing that the index being at the end of the name is more naturally) I'd be ok with just keeping it like that for now. Probably, it's just overengineering. If it's really an issue we could address it later on... -- 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]
