commit 63dcd56985516e554e8f00474b070b318a598008
author Alexey Serbin <alexey@apache.org> Mon Nov 13 21:40:01 2023 -0800
committer Abhishek Chennaka <achennaka@cloudera.com> Wed Nov 15 06:57:23 2023 +0000
tree 5704d7be9feb4b43736cb25dba1067126fd39462
parent fdaf70671ea87d2ba15dbaecf98a0a447a194834

[tablet] modernize MultiColumnWriter's code

This patch contain a few micro-optimizations, but it doesn't contain
any functional changes:
  * std::unique_ptr replaced raw pointers in the cfile_writers_ array,
    so no need to use the STLDeleteElements macro
  * replaced CHECK() with DCHECK() where appropriate
  * added more DCHECK() macros to enforce logical consistency
  * removed virtual base from the class and made it final
  * added tablet identifier into status objects and error messages
    to make it possible to attribute errors to particular tablets

Change-Id: Ife716bf62338ca896072ce7fce3aea9d5f5204ca
Reviewed-on: http://gerrit.cloudera.org:8080/20702
Tested-by: Alexey Serbin <alexey@apache.org>
Tested-by: Kudu Jenkins
Reviewed-by: Abhishek Chennaka <achennaka@cloudera.com>