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

   ### What changes were proposed in this pull request?
   
   Wait for a successful query through the Hive Metastore Thrift endpoint 
before starting Python integration tests. Bound readiness commands and polling 
with a deadline, and clean up the container if initialization fails.
   
   ### Why are the changes needed?
   
   The image's Hive CLI check can use an embedded metastore and report success 
while port 9083 is unavailable. The previous asyncio timeout cannot interrupt 
synchronous Docker calls and sleeps.
   
   Observed in 
[PythonIT](https://github.com/apache/gravitino/actions/runs/34436219502/job/102745773502),
 where relational-table setup and catalog-tag cleanup failed with Metastore 
connection refused.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. Only Python integration-test infrastructure changes.
   
   ### How was this patch tested?
   
   Four regression tests cover readiness retries, deadline handling, Docker 
errors, and failed-startup cleanup. All passed for 50 consecutive runs (200 
executions, with mocked Docker and clock). Black, Pylint, and spotlessApply 
passed. Full Docker PythonIT was not rerun locally.
   


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