[
https://issues.apache.org/jira/browse/TEZ-4650?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
László Bodor updated TEZ-4650:
------------------------------
Description:
Proposal is to remove the method:
[processSchedulerDescriptors|https://github.com/apache/tez/blob/0a5c7142c8ac5dff0fe6b18358bdb9eb99e2a678/tez-dag/src/main/java/org/apache/tez/dag/app/PluginManager.java#L198-L225]
It is for the purposes:
1. If, according to a variable isLocal, local mode is enabled (so uberEnabled),
checks if the task scheduler plugin descriptors contain a corresponding
*TezUber* entity: this is useless...if it's otherwise, it's a bug in
[parsePlugin|https://github.com/apache/tez/blob/0a5c7142c8ac5dff0fe6b18358bdb9eb99e2a678/tez-dag/src/main/java/org/apache/tez/dag/app/PluginManager.java#L170-L175]
method, which run previously and takes care about the same
2. If, according to a variable, containers are enabled (so tezYarnEnabled), but
there is no corresponding *TezYarn* entity in the descriptors, it magically
adds it: so this fixes a bug where parsePlugin didn't take care of the same
previously
[here|https://github.com/apache/tez/blob/0a5c7142c8ac5dff0fe6b18358bdb9eb99e2a678/tez-dag/src/main/java/org/apache/tez/dag/app/PluginManager.java#L163-L168]
was:Remove
> Remove useless logic from AM plugin management: processSchedulerDescriptors
> ---------------------------------------------------------------------------
>
> Key: TEZ-4650
> URL: https://issues.apache.org/jira/browse/TEZ-4650
> Project: Apache Tez
> Issue Type: Improvement
> Reporter: László Bodor
> Assignee: László Bodor
> Priority: Major
> Time Spent: 10m
> Remaining Estimate: 0h
>
> Proposal is to remove the method:
> [processSchedulerDescriptors|https://github.com/apache/tez/blob/0a5c7142c8ac5dff0fe6b18358bdb9eb99e2a678/tez-dag/src/main/java/org/apache/tez/dag/app/PluginManager.java#L198-L225]
> It is for the purposes:
> 1. If, according to a variable isLocal, local mode is enabled (so
> uberEnabled), checks if the task scheduler plugin descriptors contain a
> corresponding *TezUber* entity: this is useless...if it's otherwise, it's a
> bug in
> [parsePlugin|https://github.com/apache/tez/blob/0a5c7142c8ac5dff0fe6b18358bdb9eb99e2a678/tez-dag/src/main/java/org/apache/tez/dag/app/PluginManager.java#L170-L175]
> method, which run previously and takes care about the same
> 2. If, according to a variable, containers are enabled (so tezYarnEnabled),
> but there is no corresponding *TezYarn* entity in the descriptors, it
> magically adds it: so this fixes a bug where parsePlugin didn't take care of
> the same previously
> [here|https://github.com/apache/tez/blob/0a5c7142c8ac5dff0fe6b18358bdb9eb99e2a678/tez-dag/src/main/java/org/apache/tez/dag/app/PluginManager.java#L163-L168]
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
