Skip to content

Commit b42f3ff

Browse files
committed
fix build error
1 parent 718b62f commit b42f3ff

File tree

3 files changed

+9
-15
lines changed

3 files changed

+9
-15
lines changed

src/iceberg/table_requirement.cc

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,7 @@ Status AssertCurrentSchemaID::Validate(const TableMetadata* base) const {
4646
return NotImplemented("AssertCurrentTableSchemaID::Validate not implemented");
4747
}
4848

49-
Status AssertLastAssignedPartitionId::Validate(
50-
const TableMetadata* base) const {
49+
Status AssertLastAssignedPartitionId::Validate(const TableMetadata* base) const {
5150
return NotImplemented(
5251
"AssertCurrentTableLastAssignedPartitionId::Validate not implemented");
5352
}

src/iceberg/table_update.cc

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,7 @@ void UpgradeFormatVersion::ApplyTo(TableMetadataBuilder& builder) const {
4343
throw IcebergError(std::format("{} not implemented", __FUNCTION__));
4444
}
4545

46-
Status UpgradeFormatVersion::GenerateRequirements(
47-
TableUpdateContext& context) const {
46+
Status UpgradeFormatVersion::GenerateRequirements(TableUpdateContext& context) const {
4847
return NotImplemented(
4948
"UpgradeTableFormatVersion::GenerateRequirements not implemented");
5049
}
@@ -85,8 +84,7 @@ void SetDefaultPartitionSpec::ApplyTo(TableMetadataBuilder& builder) const {
8584
throw IcebergError(std::format("{} not implemented", __FUNCTION__));
8685
}
8786

88-
Status SetDefaultPartitionSpec::GenerateRequirements(
89-
TableUpdateContext& context) const {
87+
Status SetDefaultPartitionSpec::GenerateRequirements(TableUpdateContext& context) const {
9088
return NotImplemented(
9189
"SetDefaultTablePartitionSpec::GenerateRequirements not implemented");
9290
}
@@ -97,8 +95,7 @@ void RemovePartitionSpecs::ApplyTo(TableMetadataBuilder& builder) const {
9795
throw IcebergError(std::format("{} not implemented", __FUNCTION__));
9896
}
9997

100-
Status RemovePartitionSpecs::GenerateRequirements(
101-
TableUpdateContext& context) const {
98+
Status RemovePartitionSpecs::GenerateRequirements(TableUpdateContext& context) const {
10299
return NotImplemented(
103100
"RemoveTablePartitionSpecs::GenerateRequirements not implemented");
104101
}

src/iceberg/table_update.h

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -211,8 +211,7 @@ class ICEBERG_EXPORT AddSortOrder : public TableUpdate {
211211
/// \brief Represents setting the default sort order
212212
class ICEBERG_EXPORT SetDefaultSortOrder : public TableUpdate {
213213
public:
214-
explicit SetDefaultSortOrder(int32_t sort_order_id)
215-
: sort_order_id_(sort_order_id) {}
214+
explicit SetDefaultSortOrder(int32_t sort_order_id) : sort_order_id_(sort_order_id) {}
216215

217216
int32_t sort_order_id() const { return sort_order_id_; }
218217

@@ -259,8 +258,7 @@ class ICEBERG_EXPORT RemoveSnapshots : public TableUpdate {
259258
/// \brief Represents removing a snapshot reference
260259
class ICEBERG_EXPORT RemoveSnapshotRef : public TableUpdate {
261260
public:
262-
explicit RemoveSnapshotRef(std::string ref_name)
263-
: ref_name_(std::move(ref_name)) {}
261+
explicit RemoveSnapshotRef(std::string ref_name) : ref_name_(std::move(ref_name)) {}
264262

265263
const std::string& ref_name() const { return ref_name_; }
266264

@@ -276,9 +274,9 @@ class ICEBERG_EXPORT RemoveSnapshotRef : public TableUpdate {
276274
class ICEBERG_EXPORT SetSnapshotRef : public TableUpdate {
277275
public:
278276
SetSnapshotRef(std::string ref_name, int64_t snapshot_id, SnapshotRefType type,
279-
std::optional<int32_t> min_snapshots_to_keep = std::nullopt,
280-
std::optional<int64_t> max_snapshot_age_ms = std::nullopt,
281-
std::optional<int64_t> max_ref_age_ms = std::nullopt)
277+
std::optional<int32_t> min_snapshots_to_keep = std::nullopt,
278+
std::optional<int64_t> max_snapshot_age_ms = std::nullopt,
279+
std::optional<int64_t> max_ref_age_ms = std::nullopt)
282280
: ref_name_(std::move(ref_name)),
283281
snapshot_id_(snapshot_id),
284282
type_(type),

0 commit comments

Comments
 (0)