Hello Grant Henke,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/14431
to review the following change.
Change subject: CapturingLogAppender: synchronize access to captured log text
......................................................................
CapturingLogAppender: synchronize access to captured log text
I have no explanation for how this could happen, but the test failure I'm
looking at speaks for itself:
1) testSessionOnceClosed(org.apache.kudu.client.TestKuduClient)
java.lang.StringIndexOutOfBoundsException: String index out of range: 419
at java.lang.String.<init>(String.java:205)
at java.lang.StringBuilder.toString(StringBuilder.java:407)
at
org.apache.kudu.test.CapturingLogAppender.getAppendedText(CapturingLogAppender.java:72)
at
org.apache.kudu.client.TestKuduClient.testSessionOnceClosed(TestKuduClient.java:1231)
Change-Id: I84a5d0775cba5aa1d9df5484b5e9e621c908d42d
---
M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala
M
java/kudu-test-utils/src/main/java/org/apache/kudu/test/CapturingLogAppender.java
2 files changed, 12 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/31/14431/1
--
To view, visit http://gerrit.cloudera.org:8080/14431
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I84a5d0775cba5aa1d9df5484b5e9e621c908d42d
Gerrit-Change-Number: 14431
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
