commit c39e72a98e124e5a12080bac42105b996594d806
author Alexey Serbin <alexey@apache.org> Tue Apr 30 21:33:12 2024 -0700
committer Abhishek Chennaka <achennaka@cloudera.com> Thu May 02 21:36:20 2024 +0000
tree 93a11818f850fccba1147ef684b169058a791943
parent fdb42489d76098eeca371ac57ae02ea176438c64

[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
Reviewed-on: http://gerrit.cloudera.org:8080/21381
Tested-by: Alexey Serbin <alexey@apache.org>
Reviewed-by: Abhishek Chennaka <achennaka@cloudera.com>