KNOX-1608: Remove gateway-adapter module

[arunk-kumar]

Jira Link

KNOX-1608 - Remove gateway-adapter module

What changes were proposed in this pull request?

As outlined in KNOX-1608, the deprecated gateway-adapter module has been completely unhooked from the parent and release build lifecycles, and its directory structure has been completely purged from the codebase.

This module was originally retained as a compatibility bridge for old classpaths (org.apache.hadoop.gateway) during historical package migrations and is no longer required for modern deployment targets.

(Please fill in changes proposed in this fix)

How was this patch tested?

The patch was validated via full workspace compilation from the root directory to confirm that downstream packaging profiles (gateway-release, gateway-docker) compile cleanly without the module:

mvn clean install -DskipTests -Dspotbugs.skip=true

The local build completes successfully with a green BUILD SUCCESS flag.

Additionally, a local unit test suite was executed. Flaky tests and environment-specific constraints were observed (notably JaCoCo bytecode mismatches with local JDK 24 and race conditions on high-speed Apple Silicon CPUs), but all core modules and non-environmental test trees compile and validate cleanly.

Integration Tests

N/A (Codebase technical debt cleanup; no new business logic or operational endpoints introduced).

UI changes

N/A

Please review Knox Contributing Process before opening a pull request.

[github-actions]

Test Results

28 tests   28 ✅  3s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 93c366e.

[smolnar82]

LGTM.

@hanicz - Can you please take a look?

[hanicz]

@arunk-kumar Thank you for picking up this task.

• Please rebase the changes you are 14 commits behind master.
• gateway-admin-ui/admin-ui/app/provider-config-wizard/ldap-provider-config.ts still has two org.apache.hadoop references please fix those.

[smolnar82]

@arunk-kumar Thank you for picking up this task.

• Please rebase the changes you are 14 commits behind master.
• gateway-admin-ui/admin-ui/app/provider-config-wizard/ldap-provider-config.ts still has two org.apache.hadoop references please fix those.

Nice catch, @hanicz !

I also found a couple more references to the old package name:

$ git status
On branch review-pr-1263
nothing to commit, working tree clean

$ grep -R "org.apache.hadoop.gateway" ./*
./CHANGES:    * [KNOX-1553] - Update documentation to reference org.apache.knox instead of org.apache.hadoop.gateway
./build-tools/src/main/resources/build-tools/spotbugs-filter.xml:    <Class name="~org.apache.hadoop.gateway..*" />
./gateway-admin-ui/admin-ui/app/provider-config-wizard/ldap-provider-config.ts:            'org.apache.hadoop.gateway.shirorealm.KnoxLdapRealm');
./gateway-admin-ui/admin-ui/app/provider-config-wizard/ldap-provider-config.ts:            'org.apache.hadoop.gateway.shirorealm.KnoxLdapContextFactory');
./gateway-server/src/test/resources/org/apache/knox/gateway/topology/xml/TopologyRulesModuleTest/topology-with-dispatch-parameters.xml:            <classname>org.apache.hadoop.gateway.hbase.HBaseDispatch</classname>
./gateway-server/src/test/resources/org/apache/knox/gateway/topology/xml/TopologyRulesModuleTest/topology-with-dispatch.xml:            <classname>org.apache.hadoop.gateway.hbase.HBaseDispatch</classname>
./gateway-server/src/test/java/org/apache/knox/gateway/topology/simple/ProviderConfigurationParserTest.java:    "        \"main.ldapContextFactory\":\"org.apache.hadoop.gateway.shirorealm.KnoxLdapContextFactory\",\n" +
./gateway-server/src/test/java/org/apache/knox/gateway/topology/simple/ProviderConfigurationParserTest.java:    "        \"main.ldapRealm\":\"org.apache.hadoop.gateway.shirorealm.KnoxLdapRealm\",\n" +
./gateway-server/src/test/java/org/apache/knox/gateway/topology/simple/ProviderConfigurationParserTest.java:        "      main.ldapRealm: org.apache.hadoop.gateway.shirorealm.KnoxLdapRealm\n" +
./gateway-server/src/test/java/org/apache/knox/gateway/topology/simple/ProviderConfigurationParserTest.java:        "      main.ldapContextFactory: org.apache.hadoop.gateway.shirorealm.KnoxLdapContextFactory\n" +
./gateway-server/src/test/java/org/apache/knox/gateway/topology/simple/ProviderConfigurationParserTest.java:        assertEquals(params.get("main.ldapRealm"), "org.apache.hadoop.gateway.shirorealm.KnoxLdapRealm");
./gateway-server/src/test/java/org/apache/knox/gateway/topology/simple/ProviderConfigurationParserTest.java:        assertEquals(params.get("main.ldapContextFactory"), "org.apache.hadoop.gateway.shirorealm.KnoxLdapContextFactory");
./gateway-server/src/test/java/org/apache/knox/gateway/topology/xml/TopologyRulesModuleTest.java:    assertThat( dispatch.getClassName(), is("org.apache.hadoop.gateway.hbase.HBaseDispatch") );
./gateway-server/src/test/java/org/apache/knox/gateway/topology/xml/TopologyRulesModuleTest.java:    assertThat( dispatch.getClassName(), is("org.apache.hadoop.gateway.hbase.HBaseDispatch") );
./gateway-test/src/test/resources/org/apache/knox/gateway/GatewayBasicFuncTest/changes.txt: 62. HADOOP-1504.  Fix terminate-org.apache.hadoop-gateway script in contrib/ec2
./gateway-test/src/test/resources/org/apache/knox/gateway/GatewayBasicFuncTest/changes.txt:132. HADOOP-1635.  Remove hardcoded keypair name and fix launch-org.apache.hadoop-gateway
./gateway-topology-hadoop-xml/src/test/resources/testDescriptor.xml:          authentication.param.main.ldapRealm=org.apache.hadoop.gateway.shirorealm.KnoxLdapRealm#
./gateway-topology-hadoop-xml/src/test/resources/testDelete.xml:            authentication.param.main.ldapRealm=org.apache.hadoop.gateway.shirorealm.KnoxLdapRealm#
./gateway-topology-hadoop-xml/src/test/java/org/apache/knox/gateway/topology/hadoop/xml/HadoopXmlResourceParserTest.java:    assertEquals("org.apache.hadoop.gateway.shirorealm.KnoxLdapRealm", authenticationProvider.getParams().get("main.ldapRealm"));

The only one should remain is in CHANGES.

[moresandeep]

Thanks @arunk-kumar! can I ask you the motivation behind this change? I would also recommend sending an email on the Knox dev mailing list discussing the change before we make this change (discuss thread). It is unlikely that someone might be using these packages but it is a breaking change and we should let people know before we make these changes.

[smolnar82]

Thanks @arunk-kumar! can I ask you the motivation behind this change? I would also recommend sending an email on the Knox dev mailing list discussing the change before we make this change (discuss thread). It is unlikely that someone might be using these packages but it is a breaking change and we should let people know before we make these changes.

@arunk-kumar contacted me before he submitted his change and I assured this is ok as we planned to remove this module even earlier, but 3.0.0 will be a breaking change anyway due to the JDK 17 support.

Thanks, @moresandeep, for bringing this up, this was a 100% valid point!