jason810496 commented on code in PR #68694:
URL: https://github.com/apache/airflow/pull/68694#discussion_r3442821135
##########
airflow-core/docs/authoring-and-scheduling/language-sdks/index.rst:
##########
@@ -147,6 +148,14 @@ Coordinators are registered in ``airflow.cfg`` (or via
environment variables) un
of each coordinator (e.g. :ref:`java-sdk/coordinator-config` for
:class:`~airflow.sdk.coordinators.java.JavaCoordinator`).
+ ``extra`` is an optional object for any additional information you want to
associate with a
+ coordinator without coupling it to the coordinator instance. The
coordinator itself never
+ receives it; other components read it as needed. For example,
KubernetesExecutor reads
+ ``extra.pod_template_file`` to launch a queue's worker pod from a specific
pod template, and
+ ``extra.worker_container_repository`` + ``extra.worker_container_tag`` to
override that queue's
+ worker base image (both keys are required), e.g. an image that bundles the
JVM for a Java
+ coordinator.
+
Review Comment:
The docs and config template changes belong on the core side rather than the
provider side, so I'd prefer to keep the docs as-is in this PR. The
provider-side PR (#68713) is expected to merge after the 3.3 release, which
means if we move the extra field docs there, they won't make it into 3.3.
Keeping them here ensures the documentation ships with the release.
--
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]
