apache / gravitino

World's most powerful open data catalog for building a high-performance, geo-distributed and federated metadata lake.

https://gravitino.apache.org

Apache License 2.0

959 stars 302 forks source link

[Subtask] Fix missing-timeout Pylint rule for client-python #3765

Open noidname01 opened 4 months ago

noidname01 commented 4 months ago

Describe the subtask

Define a reasonable timeout for request.get to prevent program from hanging indefinitely

Parent issue

3560

adheeshsharma19 commented 4 months ago

Hi, can I work on this ? Also, which file under client-python requires timeout to be added.

noidname01 commented 4 months ago

@adheeshsharma19 Sure, you're welcome.😊 It's in tests/integration/integration_test_env.py, actually you can remove missing-timeout from disable section in pylintrc and run ./gradlew clients:client-python:pylint, it should log where the problem is.

adheeshsharma19 commented 4 months ago

Thanks

adheeshsharma19 commented 4 months ago

Have raised a PR :https://github.com/datastrato/gravitino/pull/3810