Michael Berman 2013-09-13, 20:03
Is anyone else seeing issues in this test (on master HEAD as of an hour
ago)? Inspecting the code, I can't see how it would ever pass, but since
Jenkins is green, I must be missing something...
In MiniAccumuloCluster:371, config.shouldRunGC() is always false, so
gcProcess is null.
MiniAccumuloCluster.getProcesses() always adds its GC to its result map,
even though in this case it's null.
RestartIT.restartMasterRecovery() calls MiniAccumuloCluster.killProcess()
on every reference returned by getProcesses, and ProcessReference.equals()
will NPE if it's wrapping a null process.
It seems like the fix should be to exclude the GC from getProcesses if it
doesn't exist, but I'm hesitant to make this change until I understand why
I'm the only one who sees this issue.
Michael Berman 2013-09-13, 20:18
Billie Rinaldi 2013-09-13, 21:27
Michael Berman 2013-09-13, 21:29
Michael Berman 2013-09-13, 21:36
Billie Rinaldi 2013-09-13, 21:57
Michael Berman 2013-09-13, 22:14
John Vines 2013-09-13, 21:35
Sean Busbey 2013-09-13, 22:20
Mike Drob 2013-09-13, 23:49
Christopher 2013-09-17, 21:18
Josh Elser 2013-09-17, 21:31
Josh Elser 2013-09-14, 02:00