Till Westmann has posted comments on this change.

Change subject: Add HTTP API to Collect Threaddumps / Misc
......................................................................


Patch Set 5:

Looks good!

However, testing it I saw

java.lang.NullPointerException
        at 
org.apache.hyracks.ipc.api.RPCInterface.deliverIncomingMessage(RPCInterface.java:57)
        at 
org.apache.hyracks.ipc.impl.IPCSystem.deliverIncomingMessage(IPCSystem.java:95)
        at 
org.apache.hyracks.ipc.impl.IPCHandle.processIncomingMessages(IPCHandle.java:165)
        at 
org.apache.hyracks.ipc.impl.IPCConnectionManager$NetworkThread.run(IPCConnectionManager.java:273)

on the console when moving from "admin/cluster" to the CC's thread dump. And 
after this I also get the same exception repeatedly when moving back (in the 
browser) to the "admin/cluster" page and then going to one of the NC's thread 
dumps.
Moving from the "admin/cluster" page to the NC's thread dumps (without going to 
the CC"s thread dump first) shows no issues.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I536606a1fbc394c6c70bb8ce14791cc411691617
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <mb...@apache.org>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-HasComments: No

Reply via email to