Michael Blow has posted comments on this change. Change subject: Make Xmx small on 32bit JVM ......................................................................
Patch Set 9: Code-Review+2 (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/1521/8/hyracks-fullstack/hyracks/hyracks-control/hyracks-nc-service/src/main/java/org/apache/hyracks/control/nc/service/NCService.java File hyracks-fullstack/hyracks/hyracks-control/hyracks-nc-service/src/main/java/org/apache/hyracks/control/nc/service/NCService.java: PS8, Line 128: (proportionalRamSize <= 1024 ? proportionalRamSize : 1024) > There's no method call for Math.min(). I'm a little sad we didn't put back Math.min() since we spun another patch... -- To view, visit https://asterix-gerrit.ics.uci.edu/1521 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I025174fc2ca53e8d15ed53fac31b43bea3ddf281 Gerrit-PatchSet: 9 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Ian Maxon <ima...@apache.org> Gerrit-Reviewer: Ian Maxon <ima...@apache.org> Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Michael Blow <mb...@apache.org> Gerrit-Reviewer: Xikui Wang <xkk...@gmail.com> Gerrit-Reviewer: Yingyi Bu <buyin...@gmail.com> Gerrit-HasComments: Yes
