xloya commented on code in PR #3876: URL: https://github.com/apache/gravitino/pull/3876#discussion_r1663964102
########## clients/client-python/tests/integration/hdfs_container.py: ########## @@ -0,0 +1,143 @@ +""" +Copyright 2024 Datastrato Pvt Ltd. +This software is licensed under the Apache License version 2. +""" + +import asyncio +import logging +import os +import time + +import docker +from docker import types as tp +from docker.errors import NotFound + +from gravitino.exceptions.gravitino_runtime_exception import GravitinoRuntimeException + +logger = logging.getLogger(__name__) + + +async def check_hdfs_status(hive_container): Review Comment: Currently, there is no logic to check whether Hive is ready, so I am afraid that using Hive Container may cause misunderstanding to others. My suggestion is that if we need to support Hive integration tests in Python in the future, someone can refactor this and add Hive status check logic. -- 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]
