[
https://issues.apache.org/jira/browse/GOBBLIN-2176?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Abhishek Jain updated GOBBLIN-2176:
-----------------------------------
Description:
Currently in
[MRJobLauncher|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/mapreduce/MRJobLauncher.java#L833]
as well as
[AbstractJobLauncher|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/AbstractJobLauncher.java#L222C7-L222C30]
we set default authenticator if the job contains the property
"job.default.authenticator.class"
This behaviour doesn't exist in Temporal worker containers because we don't
extend from AbstractJobLauncher or set the authenticator explicitly.
The authenticator is set in Temporal AM however, since we extend
AbstractJobLauncher in GobblinTemporalJobLauncher (through GobblinJobLauncher)
and the call to set default authenticator is made in the
[constructor|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/AbstractJobLauncher.java#L222C7-L222C30]
of AbstractJobLauncher.
was:
Currently in
[MRJobLauncher|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/mapreduce/MRJobLauncher.java#L833]
as well as
[AbstractJobLauncher||https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/AbstractJobLauncher.java#L222C7-L222C30]
we set default authenticator if the job contains the property
"job.default.authenticator.class"
This behaviour doesn't exist in Temporal worker containers because we don't
extend from AbstractJobLauncher or set the authenticator explicitly.
The authenticator is set in Temporal AM however, since we extend
AbstractJobLauncher in GobblinTemporalJobLauncher (through GobblinJobLauncher)
and the call to set default authenticator is made in the
[constructor|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/AbstractJobLauncher.java#L222C7-L222C30]
of AbstractJobLauncher.
> Set default authenticator in Temporal containers
> ------------------------------------------------
>
> Key: GOBBLIN-2176
> URL: https://issues.apache.org/jira/browse/GOBBLIN-2176
> Project: Apache Gobblin
> Issue Type: Bug
> Components: gobblin-service
> Reporter: Abhishek Jain
> Priority: Major
>
> Currently in
> [MRJobLauncher|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/mapreduce/MRJobLauncher.java#L833]
> as well as
> [AbstractJobLauncher|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/AbstractJobLauncher.java#L222C7-L222C30]
> we set default authenticator if the job contains the property
> "job.default.authenticator.class"
>
> This behaviour doesn't exist in Temporal worker containers because we don't
> extend from AbstractJobLauncher or set the authenticator explicitly.
>
> The authenticator is set in Temporal AM however, since we extend
> AbstractJobLauncher in GobblinTemporalJobLauncher (through
> GobblinJobLauncher) and the call to set default authenticator is made in the
> [constructor|https://github.com/apache/gobblin/blob/e5d897edaee391d05a55e6ac8a420e3416fef6d9/gobblin-runtime/src/main/java/org/apache/gobblin/runtime/AbstractJobLauncher.java#L222C7-L222C30]
> of AbstractJobLauncher.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
