-
Notifications
You must be signed in to change notification settings - Fork 410
Closed
Labels
component/storagefeature/developingseverity/minortype/bugThe issue is confirmed as a bug.The issue is confirmed as a bug.
Description
dbms/src/Storages/KVStore/MultiRaft/RegionData.cpp:283:21: error: call to constructor of 'RegionDataMemDiff' is ambiguous
283 | recordMemChange(RegionDataMemDiff{-cf_data_size, -decoded_data_size});
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:32:5: note: candidate constructor
32 | RegionDataMemDiff(Type payload_, Type decoded_)
| ^
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:36:5: note: candidate constructor
36 | RegionDataMemDiff(UInt64 payload_, UInt64 decoded_)
| ^
dbms/src/Storages/KVStore/MultiRaft/RegionData.cpp:291:23: error: call to constructor of 'RegionDataMemDiff' is ambiguous
291 | updateMemoryUsage(RegionDataMemDiff{rhs.cf_data_size, rhs.decoded_data_size});
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:32:5: note: candidate constructor
32 | RegionDataMemDiff(Type payload_, Type decoded_)
| ^
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:36:5: note: candidate constructor
36 | RegionDataMemDiff(UInt64 payload_, UInt64 decoded_)
| ^
dbms/src/Storages/KVStore/MultiRaft/RegionData.cpp:359:21: error: call to constructor of 'RegionDataMemDiff' is ambiguous
359 | recordMemChange(RegionDataMemDiff{-cf_data_size, 0});
| ^ ~~~~~~~~~~~~~~~~~~
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:32:5: note: candidate constructor
32 | RegionDataMemDiff(Type payload_, Type decoded_)
| ^
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:36:5: note: candidate constructor
36 | RegionDataMemDiff(UInt64 payload_, UInt64 decoded_)
| ^
dbms/src/Storages/KVStore/MultiRaft/RegionData.cpp:360:23: error: call to constructor of 'RegionDataMemDiff' is ambiguous
360 | updateMemoryUsage(RegionDataMemDiff{-cf_data_size, 0});
| ^ ~~~~~~~~~~~~~~~~~~
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:32:5: note: candidate constructor
32 | RegionDataMemDiff(Type payload_, Type decoded_)
| ^
dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h:36:5: note: candidate constructor
36 | RegionDataMemDiff(UInt64 payload_, UInt64 decoded_)
| ^
4 errors generated.
[155/1049] Building CXX object dbms/src/Storages/KVStore/CMakeFiles/kvstore.dir/MultiRaft/Disagg/CheckpointIngestInfo.cpp.o
ninja: build stopped: subcommand failed.
tiflash/dbms/src/Storages/KVStore/MultiRaft/RegionCFDataBase.h
Lines 26 to 43 in d27d7c0
| struct RegionDataMemDiff | |
| { | |
| using Type = Int64; | |
| Type payload; | |
| Type decoded; | |
| RegionDataMemDiff(Type payload_, Type decoded_) | |
| : payload(payload_) | |
| , decoded(decoded_) | |
| {} | |
| RegionDataMemDiff(UInt64 payload_, UInt64 decoded_) | |
| : payload(static_cast<Type>(payload_)) | |
| , decoded(static_cast<Type>(decoded_)) | |
| {} | |
| RegionDataMemDiff() | |
| : payload(0) | |
| , decoded(0) | |
| {} |
These two function overloads are too similar...
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
component/storagefeature/developingseverity/minortype/bugThe issue is confirmed as a bug.The issue is confirmed as a bug.