helix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hu...@apache.org
Subject [helix] 24/43: Fix InvalidRoutingData error message in tests (#821)
Date Tue, 17 Mar 2020 00:03:55 GMT
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

commit 388d0c0fbf2761443c7358bfded28b1bcefe9e2f
Author: Neal Sun <nealsun.0428@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);


Mime
View raw message