SamWheating commented on pull request #15714:
URL: https://github.com/apache/airflow/pull/15714#issuecomment-841811609
Hey @alippai, thanks for bringing this up. I am in agreement with Jarek's
comment and don't expect this to introduce any new issues. However if there's
something I've missed then I am happy to reevaluate these changes.
In general, if you all feel that this change is risky or too opinionated (as
we're potentially overriding people's mySQL configuration without making it
super clear), we could revert and approach this problem differently, either by:
- Mentioning this in the docs and telling people to set this isolation at
their database level.
- Moving the `isolation_level` out to a config option, similar to the
connection pooling parameters.
Thoughts?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
