pocozh commented on code in PR #5595:
URL: https://github.com/apache/inlong/pull/5595#discussion_r959284814
##########
inlong-common/src/main/java/org/apache/inlong/common/pojo/dataproxy/DataProxyNodeResponse.java:
##########
@@ -52,4 +52,8 @@ public class DataProxyNodeResponse {
*/
private List<DataProxyNodeInfo> nodeList;
+ private String loadBalance;
Review Comment:
These two configuration parameters should be added in `ProxyClientConfig`,
instead of `DataProxyNodeResponse` which is the meta data from
`inlong-manager`. Let user to choose different load balancing strategy while
initializing `ProxyClientConfig`. What's more, it is better to use `enum` type
to define `loadBalance`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
