diff --git a/src/iceberg/metadata_columns.cc b/src/iceberg/metadata_columns.cc index e00bc1661..c41191560 100644 --- a/src/iceberg/metadata_columns.cc +++ b/src/iceberg/metadata_columns.cc @@ -58,12 +58,11 @@ const std::set& MetadataColumns::MetadataFieldIds() { } bool MetadataColumns::IsMetadataColumn(std::string_view name) { - return name == kPartitionColumnName || - GetMetadataColumnMap().find(name) != GetMetadataColumnMap().end(); + return name == kPartitionColumnName || GetMetadataColumnMap().contains(name); } bool MetadataColumns::IsMetadataColumn(int32_t id) { - return GetMetadataFieldIdSet().find(id) != GetMetadataFieldIdSet().end(); + return GetMetadataFieldIdSet().contains(id); } bool MetadataColumns::IsRowLineageColumn(int32_t id) { diff --git a/src/iceberg/table_metadata.cc b/src/iceberg/table_metadata.cc index 0763c4fe6..c9b78ae7e 100644 --- a/src/iceberg/table_metadata.cc +++ b/src/iceberg/table_metadata.cc @@ -792,7 +792,7 @@ Status TableMetadataBuilder::Impl::SetDefaultSortOrder(int32_t order_id) { Result TableMetadataBuilder::Impl::AddSortOrder(const SortOrder& order) { int32_t new_order_id = ReuseOrCreateNewSortOrderId(order); - if (sort_orders_by_id_.find(new_order_id) != sort_orders_by_id_.end()) { + if (sort_orders_by_id_.contains(new_order_id)) { // update last_added_order_id if the order was added in this set of changes (since it // is now the last) bool is_new_order = diff --git a/src/iceberg/util/content_file_util.cc b/src/iceberg/util/content_file_util.cc index 23eeaec8d..fb6cc555d 100644 --- a/src/iceberg/util/content_file_util.cc +++ b/src/iceberg/util/content_file_util.cc @@ -105,7 +105,7 @@ template void DropUnselectedColumnStats(std::map& map, const std::unordered_set& columns) { for (auto it = map.begin(); it != map.end();) { - if (columns.find(it->first) == columns.end()) { + if (!columns.contains(it->first)) { it = map.erase(it); } else { ++it; diff --git a/src/iceberg/util/struct_like_set.cc b/src/iceberg/util/struct_like_set.cc index fe622c8e5..eccf80def 100644 --- a/src/iceberg/util/struct_like_set.cc +++ b/src/iceberg/util/struct_like_set.cc @@ -584,7 +584,7 @@ Result StructLikeSet::Contains(const StructLike& row) const { if constexpr (kValidate) { ICEBERG_RETURN_UNEXPECTED(ValidateRowAgainstTypes(row, field_types_)); } - return set_.find(row) != set_.end(); + return set_.contains(row); } template