[helix] branch zooscalability updated: Fix InvalidRoutingData error message in tests (#821)

[hulee]

This is an automated email from the ASF dual-hosted git repository.

hulee pushed a commit to branch zooscalability
in repository https://gitbox.apache.org/repos/asf/helix.git


The following commit(s) were added to refs/heads/zooscalability by this push:
     new ef601a1  Fix InvalidRoutingData error message in tests (#821)
ef601a1 is described below

commit ef601a19b389ff985a596b653a304056974d9983
Author: Neal Sun <nealsun.0...@gmail.com>
AuthorDate: Wed Feb 26 17:24:22 2020 -0800

    Fix InvalidRoutingData error message in tests (#821)
    
    
    This PR aims to fix the InvalidRoutingDataException error message that 
shows up in tests. The source of this error was due to two test cases in 
TestZkMetadataStoreDirectory, in which both of the test cases inserted 
"/a/b/c/d/e" directly into the same set of routing data. This renders the 
routing data invalid because one sharding key is pointing to two realms.
    This was not caught at insertion because the insertion was done directly 
instead of through a dedicated API (because the code in question is a test 
case). The tests didn't fail because they relied on the raw routing data for 
correctness. The construction of MetadataStoreRoutingData failed "correctly" as 
can be seen from the error messages, however the construction failure was after 
raw routing data being fetched, therefore making the situation undetected.
---
 .../apache/helix/rest/metadatastore/TestZkMetadataStoreDirectory.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/helix-rest/src/test/java/org/apache/helix/rest/metadatastore/TestZkMetadataStoreDirectory.java
 
b/helix-rest/src/test/java/org/apache/helix/rest/metadatastore/TestZkMetadataStoreDirectory.java
index 6fe5f32..00a328f 100644
--- 
a/helix-rest/src/test/java/org/apache/helix/rest/metadatastore/TestZkMetadataStoreDirectory.java
+++ 
b/helix-rest/src/test/java/org/apache/helix/rest/metadatastore/TestZkMetadataStoreDirectory.java
@@ -253,7 +253,7 @@ public class TestZkMetadataStoreDirectory extends 
AbstractTestClass {
     // Since there was a child change callback, make sure data change works on 
the new child (realm) as well by adding a key
     // This tests removing all subscriptions and subscribing with new children 
list
     // For all namespaces (Routing ZKs), add an extra sharding key to 
TEST_REALM_3
-    String newKey = "/a/b/c/d/e";
+    String newKey = "/x/y/z";
     _zkList.forEach(zk -> {
       ZNRecord znRecord = ZK_SERVER_MAP.get(zk).getZkClient()
           .readData(MetadataStoreRoutingConstants.ROUTING_DATA_PATH + "/" + 
TEST_REALM_3);