[GitHub] [airflow] potiuk commented on pull request #23715: breeze test parallel

[GitBox <gi...@apache.org>]

potiuk commented on PR #23715:
URL: https://github.com/apache/airflow/pull/23715#issuecomment-1227633781

   > Here why do you set the default to 4 if the env `output_file_for_recording` is available? If I am right in thinking, that env points to the `output_command.svg` file (print help in command as svg using rich) How does it influence the number of cpu count to use?
   
   :smile: Very good question :)
   
   The reason is very simple. The `recording` case is there to produce `breeze` documentation images, and those images are generated when set of commands or arguments change. This means that we have to make sure that no matter on what machine we run it - when recording images are prepared we always get THE SAME set o commands and flags.
   
   Today - when you run this command on 4 CPU machine, the command will have 4 as default, when you run it on 8 CPU machine, the default is 8. So the set of arguments is different, depending on which machine you run it - and generated "documentation" will slightly differ and we need to make sure that no matter if you or me generate the docs, it will be the same.  That's why when we are in "Recording" mode, some variable parts of commands and args are fixed to be independent of the environment.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org