o-nikolas commented on code in PR #27067:
URL: https://github.com/apache/airflow/pull/27067#discussion_r1023280146
##########
airflow/dag_processing/manager.py:
##########
@@ -1129,7 +1129,6 @@ def _kill_timed_out_processors(self):
)
Stats.decr('dag_processing.processes')
Stats.incr('dag_processing.processor_timeouts')
Review Comment:
@potiuk
> is that clearer now where I am coming from ? Is that something that we can
agree on as a direction :) ?
Yupp, and creating a dev email list discussion seems like a reasonable next
step to me. Perhaps the approach we decide on should even be enshrined in an
AIP (even though that might be unconventional).
Ultimately, the most important things to me are to 1) make a group/community
decision on the direction (whether that's strict SemVer, our own approach,
something in between) 2) document that direction as explicitly and detailed as
possible, then 3) stay consistent with whatever we decide.
2 and 3 combined will reduce developer overhead and long discussions each
time this question comes up (since it will always come up a lot).
--
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]
