xBis7 commented on PR #54103: URL: https://github.com/apache/airflow/pull/54103#issuecomment-3417939313
@Asquator This patch improves the existing implementation and increases performance. From all the discussions so far, the way I see it, there is no nice way to extend the current approach to the point of accounting for all the edge cases. So, we have to implement a new selection algo from scratch. We can't do that overnight, so why not accept the improvement for now while looking for a new way? The issues is that the algo has to be performant (at least as performant as the current code if not better) and simple enough for people to understand it and be able to debug it. This is open-source and maintainability comes first. Performance can later be improved as long as the next guy understands the code and can work on 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
