Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/5956

to look at the new patch set (#2).

Change subject: [java client] Limit the message size in Status
......................................................................

[java client] Limit the message size in Status

We've seen cases where RPC traces that are getting logged are MBs in size.
Even if we limit the number of traces, the messages themselves can get quite 
big.

This fix is only for the Java client since it's the only client with tracing at
the moment. Ideally we'd limit this also in the server but it's still desirable
to have the clients protect themselves.

With the limits in place, we can have a maximum of 100 * 256 = 25KB of messages
in a single RPC's trace.

Change-Id: I5868da9ee6b4befd1dc863570b8e251369213ffa
---
M java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java
M java/kudu-client/src/main/java/org/apache/kudu/client/Status.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestStatus.java
3 files changed, 88 insertions(+), 39 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/5956/2
-- 
To view, visit http://gerrit.cloudera.org:8080/5956
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5868da9ee6b4befd1dc863570b8e251369213ffa
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to