You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
### What changes were proposed in this pull request?
1. make RPC_SERVER_TYPE an enum
2. remove ServerType definition in various places
### Why are the changes needed?
This resolves#782
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Existing UTs
xianjingfeng
pushed a commit
to xianjingfeng/incubator-uniffle
that referenced
this issue
Apr 5, 2023
…#783)
### What changes were proposed in this pull request?
1. make RPC_SERVER_TYPE an enum
2. remove ServerType definition in various places
### Why are the changes needed?
This resolvesapache#782
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Existing UTs
There are multiple definitions of ServerType enums, let's refactor that in a new pr.
Originally posted by @advancedxy in #777 (comment)
The text was updated successfully, but these errors were encountered: