o-nikolas commented on code in PR #27541:
URL: https://github.com/apache/airflow/pull/27541#discussion_r1015814625


##########
airflow/config_templates/default_airflow.cfg:
##########
@@ -524,9 +524,6 @@ username =
 password =
 
 [operators]
-# The default owner assigned to each new operator, unless
-# provided explicitly or passed via ``default_args``
-default_owner = airflow

Review Comment:
   Why are we deprecating this at all? I actually like the proposal of not 
using "airflow" as the default, but I don't see why that means we have to also 
deprecate the ability for folks to set a default owner at the config level at 
all? Perhaps they have a default that makes sense for their workflow or 
organizational structure.
   
   IMHO, this config option should just no longer be a "mandatory" value, but 
still available to use.



-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to