abdullah alamoudi has posted comments on this change. Change subject: Remove Server Context ......................................................................
Patch Set 1: too many contexts = confusion. not only for existing developers but also for new ones. and those are wrappers with unused methods. right now, on CC, we have: 1. ServerContext (wrapper for baseDir) 2. ICCContext (wrapper for unused cluster topology?) 3. ICCApplicationContext (the only used one) We can't add contexts for potential future use. Once we have the need for additional contexts, we can add them. -- To view, visit https://asterix-gerrit.ics.uci.edu/1320 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4261cf44410b498e84ffcac5e217281a306a7f03 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: No
