HDDS-14385. Incorrect net.topology.node.switch.mapping.impl default value

[Russole]

What changes were proposed in this pull request?

• Fixed the default value of net.topology.node.switch.mapping.impl in ozone-default.xml
• Updated the default to org.apache.hadoop.net.TableMapping to match the actual behavior of OM and SCM

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-14385

How was this patch tested?

All CI checks passed.
https://github.com/Russole/ozone/actions/runs/20881195253

[sreejasahithi]

Thanks @Russole for fixing this,
LGTM

[sarvekshayr]

Thanks @Russole for the fix.

The description is about invoking a script, which is specific to ScriptBasedMapping, not TableMapping.

[sreejasahithi]

Thanks @sarvekshayr for identifying the description mismatch , @Russole please update the description.

[Russole]

Thanks @sreejasahithi and @sarvekshayr for the review. I’ve updated the patch based on the comments.

[jojochuang]

actually according to https://issues.apache.org/jira/browse/HDDS-14385

The ScriptBasedMapping is used by default for node network location resolve.

[Russole]

Thanks @jojochuang for pointing this out.
Based on the discussion so far, it seems that this PR may no longer be necessary.
Please let me know if this PR is still required.

[jojochuang]

Instead, could you update the source code to make it consistent and understandable?

ozone/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/server/StorageContainerManager.java

Line 729 in fd89481

TableMapping.class, DNSToSwitchMapping.class);

ozone/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java

Line 365 in 1dda9ab

TableMapping.class, DNSToSwitchMapping.class);

update it so the default is ScriptBasedMapping