Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9155 )
Change subject: IMPALA-6450: fix EventSequence::Start() ...................................................................... IMPALA-6450: fix EventSequence::Start() It looks like this newly-added DCHECK is being hit because of the same underlying issue as IMPALA-4631. This patch loosens the DCHECK to accept time going backward 1 tick, the same as the original workaround for IMPALA-4631. 'offset_' also isn't being used, so fix that too. Change-Id: I62317149cb8428f7d29e945a538f6cc8dd45342f Reviewed-on: http://gerrit.cloudera.org:8080/9155 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/util/runtime-profile-counters.h 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9155 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I62317149cb8428f7d29e945a538f6cc8dd45342f Gerrit-Change-Number: 9155 Gerrit-PatchSet: 5 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: anujphadke <[email protected]>
