Skip to content

Conversation

@zuston
Copy link
Member

@zuston zuston commented May 20, 2024

What changes were proposed in this pull request?

fix classCastExpection of boolean->String with yaml style remote client conf

Why are the changes needed?

Fix: #1721

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Unit tests

@zuston zuston requested a review from jerqi May 20, 2024 08:38
@codecov-commenter
Copy link

codecov-commenter commented May 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.06%. Comparing base (6f6d35a) to head (bedaf04).
Report is 25 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1722      +/-   ##
============================================
- Coverage     54.86%   54.06%   -0.80%     
- Complexity     2358     2924     +566     
============================================
  Files           368      432      +64     
  Lines         16379    23356    +6977     
  Branches       1504     2187     +683     
============================================
+ Hits           8986    12627    +3641     
- Misses         6862     9951    +3089     
- Partials        531      778     +247     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link

github-actions bot commented May 20, 2024

Test Results

 2 405 files  ±0   2 405 suites  ±0   4h 57m 20s ⏱️ -58s
   931 tests ±0     930 ✅ ±0   1 💤 ±0  0 ❌ ±0 
10 791 runs  ±0  10 777 ✅ ±0  14 💤 ±0  0 ❌ ±0 

Results for commit bedaf04. ± Comparison against base commit c3cbdec.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@jerqi jerqi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@zuston zuston merged commit d9b1d9f into apache:master May 21, 2024
@jerqi
Copy link
Contributor

jerqi commented May 21, 2024

@zuston Could you commit this to branch 0.9?

zuston added a commit that referenced this pull request May 22, 2024
…yaml style remote client conf (#1722)

### What changes were proposed in this pull request?

fix classCastExpection of boolean->String with yaml style remote client conf

### Why are the changes needed?

Fix: #1721 

### Does this PR introduce _any_ user-facing change?


No.

### How was this patch tested?

Unit tests
@zuston
Copy link
Member Author

zuston commented May 22, 2024

@zuston Could you commit this to branch 0.9?

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] ClassCastExpection of boolean -> string when getting remote client conf in coordinator

3 participants