Jeffrey F. Lukman has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9069 )

Change subject: KUDU-2126: Add conditional check to prevent unnecessary fsyncs
......................................................................


Patch Set 3:

> Quick question: did you try running this test without the fix? Does
 > the test fail?

Yes, I did. Without the fix, the test failed.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idde8b413374f43ca2ef09339f0f412208f03685e
Gerrit-Change-Number: 9069
Gerrit-PatchSet: 3
Gerrit-Owner: Jeffrey F. Lukman <[email protected]>
Gerrit-Reviewer: Jeffrey F. Lukman <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Comment-Date: Sat, 20 Jan 2018 03:50:38 +0000
Gerrit-HasComments: No

Reply via email to