uranusjr commented on PR #68548: URL: https://github.com/apache/airflow/pull/68548#issuecomment-4824420408
I’m thinking, maybe the coordinator should be NodeCoordinator instead. Go does not have a GoCoordinator but a more general ExecutableCoordinator. Java has JavaCoordinator, but that’s not (mainly) because the SDK targets Java (it doesn’t _just_ target Java but all JVM languages), but more because the command that runs the SDK bundle is called `java` (even though the command runs a JVM—it’s what it is). So NodeCoordinator arguably is more suitable than TypeScriptCoordinator since the former better describes what the coordinator actually does, on the same level as the other existing coordinators. -- 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]
