pjfanning opened a new pull request, #1910:
URL: https://github.com/apache/pekko-connectors/pull/1910

   ### Motivation
   The hbase CI job has been disabled since 2020 (inherited from alpakka), so 
the HBase connector has no integration coverage on `1.4.x`. The root cause in 
akka/alpakka#2185 was that the `harisekhon/hbase` container's region server 
registers itself in ZooKeeper under the hostname `hbase`, which the runner 
cannot resolve — all tests then time out. The 2020 image rebuild also moved the 
region server to the default ports.
   
   ### Modification
   Backport of the CI parts of #1895, deliberately **without** the docker image 
or hbase client upgrades — `1.4.x` keeps `harisekhon/hbase:1.4` and 
`hbase-shaded-client` 1.4.14, so the `.scala-steward.conf` pins and the 1.x-API 
`HBaseCapabilities` code also stay unchanged:
   - Add `scripts/hbase-server.sh` (copied from #1895; same pattern as 
`scripts/ftp-servers.sh`, since `$PRE_CMD` cannot contain shell operators): 
appends `127.0.0.1 hbase` to `/etc/hosts` and starts the `hbase` docker-compose 
service; re-enable the `hbase` entry in the CI connectors matrix.
   - Publish the HBase-default ports 16020/16030 instead of the stale 
16201/16301.
   - Add the directional flow tests for append, increment, delete and complex 
converters from #1895, adapted to the 1.4.x test style (JDK 8 idioms — 
`Arrays.asList`/`String.format` — and the 1.x `Append.add` instead of the 2.x 
`Append.addColumn`).
   
   ### Result
   The HBase connector has working CI integration tests again on `1.4.x`, with 
docker image and client versions unchanged.
   
   ### Tests
   - `sbt "hbase/Test/compile"` and `sbt "hbase/javafmtCheckAll"` — clean (JDK 
17 for tooling; test code is JDK 8 compatible)
   - `scalafmt --mode diff-ref=upstream/1.4.x` — clean
   - Integration runs in the re-enabled `connectors (hbase)` CI job; this exact 
configuration (1.4 image, 1.4.14 client, ports 16020/16030, hosts-file script) 
passed CI on main at e6e7963ba before #1895 upgraded the image/client. Local 
integration not run (requires sudo to edit `/etc/hosts`).
   - `scripts/hbase-server.sh` is copied verbatim from #1895 in this repository 
(header preserved).
   
   ### References
   Refs #1895, Refs #61, Refs https://github.com/akka/alpakka/issues/2185
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to