mchades commented on issue #9025:
URL: https://github.com/apache/gravitino/issues/9025#issuecomment-3501183672

   > I think this change makes it more portable and reliable, and uses proper 
Testcontainers APIs (container.getHost(), getMappedPort()).
   
   But I am worried that doing so may not cover the existing Trino integration 
test scenarios: Trino's integration tests will start multiple containers, and 
multiple containers need to access each other. Will using getMappedPort() lead 
to the containers being unable to access each other?


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