anishgirianish commented on PR #62343: URL: https://github.com/apache/airflow/pull/62343#issuecomment-3989390075
> Thank you for the update! > > I’m still a bit concerned about mixing the concepts of `queue`, `executor`, and `team_name`, as this might create ambiguity for users in the future. It may be better to align this with Task‑level semantics, since running a connection test on a worker is effectively another type of workload. We should ensure that the way users specify the runtime environment is clear and consistent. > > Thanks! Thanks @jason810496 ! Addressed all your comments in the latest push, renamed queue to executor across the board to avoid ambiguity with task-level queue semantics. Also raised NotImplementedError in base_executor and added skip_locked to the reaper. Would like to request you for a re-review whenever you get a chance! thank you! -- 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]
