Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16317 )

Change subject: [monotime] add MonoDelta::operator{+,-}()
......................................................................


Patch Set 2: Verified+1

LSAN issue is one we've seen, probably unrelated to this patch:


=================================================================
==24521==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 4368 byte(s) in 2 object(s) allocated from:
    #0 0x55d601 in malloc 
/home/jenkins-slave/workspace/kudu-master/2/thirdparty/src/llvm-9.0.0.src/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:145
    #1 0x7f8f7b34adb2 in CRYPTO_malloc 
(/lib/x86_64-linux-gnu/libcrypto.so.1.0.0+0x5fdb2)

SUMMARY: AddressSanitizer: 4368 byte(s) leaked in 2 allocation(s).


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibfad30af72128b4d0054993243bfc00e67b89a9e
Gerrit-Change-Number: 16317
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 10 Aug 2020 22:15:12 +0000
Gerrit-HasComments: No

Reply via email to