nevzheng opened a new pull request, #14087:
URL: https://github.com/apache/gravitino/pull/14087

   ### What changes were proposed in this pull request?
   
   - Kafka (`org.testcontainers.kafka.KafkaContainer`), LocalStack, ClickHouse 
and OceanBase (`OceanBaseCEContainer`) now wrap the official Testcontainers 
modules, as MySQL and PostgreSQL already do; images still come from 
`TestImages`.
   - Where a module's own wait covers readiness, the probe from the previous PR 
goes away; where ours checks more, it stays as an extra step.
   - Images, host names, environment, disposable-data recipes and the 
three-node ClickHouse cluster are unchanged.
   
   ### Why are the changes needed?
   
   The modules know their service's ports, defaults and readiness, so our 
wrappers keep only what is Gravitino-specific.
   
   Fix: #14002
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. Test code only.
   
   ### How was this patch tested?
   
   Unit tests for module wiring; Kafka, client-java Kafka, S3 (fileset, GVFS, 
Hive, Paimon), Trino OAuth, ClickHouse and OceanBase ITs.
   
   


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

Reply via email to