Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21381


Change subject: [tablet] fix WriteOp::ToString() for non-started ops
......................................................................

[tablet] fix WriteOp::ToString() for non-started ops

After [1] and prior to this patch, at least linked_list-test and
update_scan_delta_compact-test would fail from time to time with
DCHECK triggered like in the snippet below:

  F20240430 19:48:41.064476 2463727 monotime.cc:342] Check failed: 
t_beg.Initialized()
  *** Check failure stack trace: ***
      @     0x7f3c1472548d  google::LogMessage::Fail() at ??:0
      @     0x7f3c147290d7  google::LogMessage::SendToLog() at ??:0
      @     0x7f3c14724e8c  google::LogMessage::Flush() at ??:0
      @     0x7f3c147265c9  google::LogMessageFatal::~LogMessageFatal() at ??:0
      @     0x7f3c1564dff0  kudu::operator-() at ??:0
      @     0x7f3c1b46a0cd  kudu::tablet::WriteOp::ToString() at ??:0
      @     0x7f3c1b454416  kudu::tablet::OpDriver::ToStringUnlocked() at ??:0
      @     0x7f3c1b45436f  kudu::tablet::OpDriver::ToString() at ??:0
      @     0x7f3c1b42d154  kudu::tablet::TabletReplica::GetInFlightOps() at 
??:0
      @     0x7f3c1c121c1d  
kudu::tserver::TabletServerPathHandlers::HandleTransactionsPage() at ??:0

This is a follow-up to [1].

[1] 
https://github.com/apache/kudu/commit/ad3936521af034ffcac637f97cd8c932f6289b4f

Change-Id: Ie4cf49a72d4c7c40c6663a763e79958643fac9a4
---
M src/kudu/tablet/ops/write_op.cc
1 file changed, 13 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/81/21381/1
--
To view, visit http://gerrit.cloudera.org:8080/21381
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie4cf49a72d4c7c40c6663a763e79958643fac9a4
Gerrit-Change-Number: 21381
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to