KeDeng has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/23064


Change subject: [tablet] eliminate ABI warning by replacing std::pair with 
struct in ComputeLowerAndUpperBound
......................................................................

[tablet] eliminate ABI warning by replacing std::pair with struct in 
ComputeLowerAndUpperBound

In the Rocky Linux + ARM environment, GCC 10.1 emits a -Wpsabi warning due to
changes in parameter passing semantics for std::pair<double, double> when C++17
is enabled. The compiler note indicates that the calling convention for such
types was changed in GCC 10.0 and reverted in GCC 10.1 to match C++14 behavior.

To avoid ABI ambiguity and suppress the warning, this patch replaces the
std::pair<double, double> return type in ComputeLowerAndUpperBound() with a
custom BoundRange struct. This change improves ABI stability across platforms,
clarifies semantics by naming fields explicitly, and ensures forward
compatibility with future compiler updates.

No functional behavior is changed.

Change-Id: Iab4db585a8eaf152f1fa2884e72e38a4e7c5efa1
---
M src/kudu/tablet/compaction_policy.cc
1 file changed, 9 insertions(+), 3 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iab4db585a8eaf152f1fa2884e72e38a4e7c5efa1
Gerrit-Change-Number: 23064
Gerrit-PatchSet: 1
Gerrit-Owner: KeDeng <[email protected]>

Reply via email to