helix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hu...@apache.org
Subject [helix] branch zooscalability updated: Fix InvalidRoutingData error message in tests (#821)
Date Thu, 27 Feb 2020 01:24:33 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

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.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
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);

View raw message