jiajunwang commented on a change in pull request #1451:
URL: https://github.com/apache/helix/pull/1451#discussion_r510516235
##########
File path:
helix-rest/src/test/java/org/apache/helix/rest/server/TestClusterAccessor.java
##########
@@ -259,7 +259,15 @@ public void testUpdateConfigFields() throws IOException {
record.getSimpleFields().clear();
record.setSimpleField(key, value + "--updated");
- key = record.getListFields().keySet().iterator().next();
+ // skip INSTANCE_CAPACITY_KEYS which can be added by waged rebalancer in
global run
Review comment:
That was a guess, yeah. But the key is the current way is not the
correct solution, IMHO.
If our conclusion was wrong, that means we still don't understand the root
cause. I would say we need to clearly understand why the WAGED field was added
without any related test running. Then we can fix it in the right way.
Do you have the github environment execute order so I can reproduce it?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
