xorbitsai / xorbits

Scalable Python DS & ML, in an API compatible & lightning fast way.
https://xorbits.readthedocs.io
Apache License 2.0
1.1k stars 67 forks source link

BUG: Fix issues when connecting to a real yarn cluster #647

Closed ChengjieLi28 closed 1 year ago

ChengjieLi28 commented 1 year ago

What do these changes do?

Fixes:

Related issue number

Fixes #605

Check code requirements

codecov[bot] commented 1 year ago

Codecov Report

Merging #647 (c86764e) into main (9e789c6) will decrease coverage by 0.42%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #647      +/-   ##
==========================================
- Coverage   93.61%   93.19%   -0.42%     
==========================================
  Files        1024     1024              
  Lines       79432    79432              
  Branches    16475    16475              
==========================================
- Hits        74360    74029     -331     
- Misses       3404     3660     +256     
- Partials     1668     1743      +75     
Flag Coverage Δ
unittests 93.13% <ø> (-0.37%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

see 51 files with indirect coverage changes

ChengjieLi28 commented 1 year ago

Very welcome @smartguo to help review this PR! And Thanks very much for helping us test this PR on a real large scale yarn cluster!