[Bug]: Mix usage of column_id and column_idx in colmn_meta construction #2913
Description
Is there an existing issue for the same bug?
- I have checked the existing issues.
Version or Commit ID
Other environment information
Actual behavior and How to reproduce it
When creating instance of column meta, the constructor should be using column_idx. But in some cases column id is being used. Some possible problematic instances are:
//-------------------------------------------------------------------------------StatusNewCatalog::LoadImportedOrCompactedSegment(
.....
for (const auto &column_def : *column_defs_ptr) {
ColumnMeta column_meta(column_def->id(), *block_meta);
.....
Status NewCatalog::LoadFlushedBlock1(SegmentMeta &segment_meta, const WalBlockInfo &block_info, TxnTimeStamp checkpoint_ts) {
......
ColumnMeta column_meta(column_def->id(), block_meta);
......
-------------------------------------------------------------------------------//
Each column_meta construction should be checked
Expected behavior
No response
Additional information
No response