Skip to content

Commit a0e342f

Browse files
author
xiao.dong
committed
fix lint
1 parent c860fe2 commit a0e342f

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/iceberg/manifest_entry.h

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -297,12 +297,13 @@ struct ICEBERG_EXPORT ManifestEntry {
297297
SchemaField::MakeRequired(0, "status", iceberg::int32());
298298
inline static const SchemaField kSnapshotId =
299299
SchemaField::MakeOptional(1, "snapshot_id", iceberg::int64());
300+
inline static const int32_t kDataFileFieldId = 2;
301+
inline static const std::string kDataFileField = "data_file";
300302
inline static const SchemaField kSequenceNumber =
301303
SchemaField::MakeOptional(3, "sequence_number", iceberg::int64());
302304
inline static const SchemaField kFileSequenceNumber =
303305
SchemaField::MakeOptional(4, "file_sequence_number", iceberg::int64());
304-
inline static const int32_t kDataFileFieldId = 2;
305-
inline static const std::string kDataFileField = "data_file";
306+
306307
bool operator==(const ManifestEntry& other) const;
307308

308309
static std::shared_ptr<StructType> TypeFromPartitionType(

src/iceberg/partition_spec.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ Result<std::shared_ptr<Schema>> PartitionSpec::partition_schema() {
6565
return nullptr;
6666
}
6767
{
68-
std::lock_guard<std::mutex> lock(mutex_);
68+
std::scoped_lock<std::mutex> lock(mutex_);
6969
if (partition_schema_ != nullptr) {
7070
return partition_schema_;
7171
}
@@ -96,7 +96,7 @@ Result<std::shared_ptr<Schema>> PartitionSpec::partition_schema() {
9696
);
9797
}
9898

99-
std::lock_guard<std::mutex> lock(mutex_);
99+
std::scoped_lock<std::mutex> lock(mutex_);
100100
if (partition_schema_ == nullptr) {
101101
partition_schema_ = std::make_shared<Schema>(std::move(partition_fields));
102102
}

0 commit comments

Comments
 (0)