potiuk commented on PR #25301: URL: https://github.com/apache/airflow/pull/25301#issuecomment-1195334652
> Rather than having to do the same replacement in 4 different places might it be cleaner to set that once in what ever is setting `TEST_TYPE` (and/or introduct a `TEST_TYPE_SAFE`/`TEST_TYPE_CLEAN` variable if there are more uses that want the existing full one) I thought about it, but we would have to additionally pass this extra variable across multiple levels - from Host script (where we assign the name of docker compose project) to parallel execution script (which executes gnu paralllel) and finally to actual tes execution script (the one that is executed by gnu-parallel). It is totally not worth it especialy that we are going to replace it with new python-based breeze parallelism (as we did with other parts already). This is something that is is in progress #23715 and I hope will be finished soon. -- 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]
