search

vesoft-inc / nebula-python

Client API of Nebula Graph in Python
194 stars 76 forks source link

[v2.6.1 patch] Refactor connection pool and add more tests #251

Closed Aiee closed 1 year ago

Aiee commented 1 year ago

What type of PR is this?

What problem(s) does this PR solve?

Issue(s) number:

Description:

Address some common issues along with https://github.com/vesoft-inc/nebula-python/pull/247:

  1. Refactor code of the connection pool to simplify the implementation and print more user-friendly logs
  2. Remove unusable connection if connection Ping() failed
  3. Add automated test cases regarding the problem above

How do you solve it?

Special notes for your reviewer, ex. impact of this fix, design document, etc: