Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spark/Lightning: add reader_worker_count and reader_pool_type #3612

Merged
merged 1 commit into from Jul 22, 2022

Conversation

chongxiaoc
Copy link
Collaborator

Checklist before submitting

  • Did you read the contributor guide?
  • Did you update the docs?
  • Did you write any tests to validate this change?
  • Did you update the CHANGELOG, if this change affects users?

Description

Fixes # (issue).

Review process to land

  1. All tests and other checks must succeed.
  2. At least one member of the technical steering committee must review and approve.
  3. If any member of the technical steering committee requests changes, they must be addressed.

@chongxiaoc chongxiaoc requested a review from Tixxx July 21, 2022 21:17
@github-actions
Copy link

Unit Test Results

     893 files  +   405       893 suites  +405   10h 3m 7s ⏱️ + 2h 56m 57s
     781 tests ±       0       737 ✔️ +   106       44 💤  -    106  0 ±0 
19 061 runs  +8 479  13 666 ✔️ +6 314  5 395 💤 +2 165  0 ±0 

Results for commit 102ee5a. ± Comparison against base commit ba98ad5.

@github-actions
Copy link

Unit Test Results (with flaky tests)

  1 035 files  +     519    1 035 suites  +519   10h 44m 3s ⏱️ + 3h 33m 52s
     781 tests ±         0       737 ✔️ +   106       44 💤  -    106  0 ±0 
22 367 runs  +11 237  15 594 ✔️ +7 952  6 773 💤 +3 285  0 ±0 

Results for commit 102ee5a. ± Comparison against base commit ba98ad5.

@Tixxx
Copy link
Collaborator

Tixxx commented Jul 22, 2022

LGTM

@chongxiaoc chongxiaoc merged commit c94eb3e into horovod:master Jul 22, 2022
leewyang pushed a commit to leewyang/horovod that referenced this pull request Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants