kaxil commented on pull request #19637:
URL: https://github.com/apache/airflow/pull/19637#issuecomment-971922958


   >One thing that I really do not like here is that it forces the admin users 
to "know" about the serialization, be able to diagnose problems connected with 
it and map it mentally to the switch or command line.
   
   It won't be one size fits all. This is an escape hatch, they don't need to 
run it but if they face an issue then they can run and mitigate the issue and 
there is already a precedence for it with `sync-perm`.
   
   I can write a good enough description for it :) but I don't have a strong 
opinion if we should have that flag or not.
   
   However, I have a strong opinion that we should not force re-serialize 
during Airflow Upgrade !
   
   
   


-- 
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]


Reply via email to