Abhishek Chennaka has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20743 )

Change subject: [java] sync slow DNS resolution threshold with the C++ code
......................................................................

[java] sync slow DNS resolution threshold with the C++ code

Prior to this patch, the threshold for slow DNS resolution warning
was set to just 3 ms in the code of the Kudu Java client, which didn't
make a lot of sense if talking about non-cached DNS queries.

This patch synchronizes the warning threshold with the corresponding
threshold in the Kudu's C++ code (200 ms), and also doubles the debug
threshold, bumping up the latter from 0.5 ms to 1 ms.

Change-Id: I8c2b938b2738e1127adfdf834f0e7f0176cf35b7
Reviewed-on: http://gerrit.cloudera.org:8080/20743
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
---
M java/kudu-client/src/main/java/org/apache/kudu/util/NetUtil.java
1 file changed, 20 insertions(+), 8 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Abhishek Chennaka: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/20743
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8c2b938b2738e1127adfdf834f0e7f0176cf35b7
Gerrit-Change-Number: 20743
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <[email protected]>

Reply via email to