Adar Dembo has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/14431 )
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
Reviewed-on: http://gerrit.cloudera.org:8080/14431
Reviewed-by: Greg Solovyev <[email protected]>
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <[email protected]>
---
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(-)
Approvals:
Greg Solovyev: Looks good to me, but someone else must approve
Kudu Jenkins: Verified
Grant Henke: Looks good to me, approved
--
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: merged
Gerrit-Change-Id: I84a5d0775cba5aa1d9df5484b5e9e621c908d42d
Gerrit-Change-Number: 14431
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Greg Solovyev <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
