abdullah alamoudi has posted comments on this change.

Change subject: Remove static cc application context instance
......................................................................


Patch Set 5:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/1606/5/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/CcApplicationContext.java
File 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/CcApplicationContext.java:

PS5, Line 107:        
Logger.getLogger("org.apache.asterix").setLevel(externalProperties.getLogLevel());
             :         
Logger.getLogger("org.apache.hyracks").setLevel(externalProperties.getLogLevel());
> I am not sure. maybe it was here at some point and we lost it with rebasing
Indeed.
https://asterix-gerrit.ics.uci.edu/#/c/1606/1/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/CcApplicationContext.java


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1606
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia2e250405967ec880e7af6387aa981f39b3392c0
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Steven Jacobs <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: Yes

Reply via email to