[GitHub] [airflow] potiuk commented on issue #6975: [AIRFLOW-1467] Dynamic pooling via allowing tasks to use more than one pool slot (depending upon the need)

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

potiuk commented on issue #6975: [AIRFLOW-1467] Dynamic pooling via allowing tasks to use more than one pool slot (depending upon the need)
URL: https://github.com/apache/airflow/pull/6975#issuecomment-573879812
 
 
   Hello @lokeshlal @dimberman - I had to revert this change from master as it created two  heads in sqlalchemy migrations and we have test to detect that. The root cause was that this PR was not rebased after the second head was added (otherwise it would have been detected by failing Travis CI run - last time Travis was run 6 days ago for this PR and the other head (#6489 ) was merged 3 days ago. 
   
   It's no-one fault, it can happen in our process, but we will have to add better protection against such problems in the future. In the meantime can you please create another PR with this change, fix the duplicated heads (by making your migration based on the new head migration) and re-submit/merge the PR again.
   
   Sorry for the inconvenience @lokeshlal - I understand it was your first PR merged and you hit this - very rare - problem :(. We will do better in the future to avoid it.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services