asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yingyi Bu (Code Review)" <>
Subject Change in asterixdb[master]: Remove ICCContext
Date Sat, 29 Oct 2016 17:12:47 GMT
Yingyi Bu has posted comments on this change.

Change subject: Remove ICCContext

Patch Set 4:

File asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/util/

Line 58:         map.putAll(((ClusterControllerService) AsterixAppContextInfo.INSTANCE.getCCApplicationContext()
Still cast here.
File hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/service/

Line 39:     Map<InetAddress, Set<String>> getIpAddressNodeNameMap();
Why do we need this new interface?

The only difference with ICCContext is that now you can inline clustercontrolerinfo and ipaddresses
into the ClusterControllerService.

I don't think it's a good idea to make everything inlined into ClusterControllerService, because

(1) The class is already large.

(2) Often times, you only need to expose the necessary information, i.e., a facet to a client.
In this way, a client won't be able to call other 10+ public methods in ClusterControllerService,
which is dangerous.  Maybe it's not fully leveraged in the current codebase, but I don't think
the design is bad.
File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-cc/src/main/java/org/apache/hyracks/control/cc/

Line 357:         return ipAddressNodeNameMap;
How do you handle dynamic cluster membership now?

This seems to always return an empty map?

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I6f6a769741f14e91bcd4b970b4a022c0a453d380
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Michael Blow <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message