Skip to content

Commit 74f17e3

Browse files
author
xiao.dong
committed
fix cpplint
1 parent bbc6849 commit 74f17e3

File tree

1 file changed

+18
-6
lines changed

1 file changed

+18
-6
lines changed

src/iceberg/manifest_writer.cc

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,11 @@ Result<std::unique_ptr<ManifestWriter>> ManifestWriter::MakeV1Writer(
7171
std::make_unique<ManifestEntryAdapterV1>(snapshot_id, std::move(partition_spec));
7272
ICEBERG_RETURN_UNEXPECTED(adapter->Init());
7373
ICEBERG_RETURN_UNEXPECTED(adapter->StartAppending());
74+
75+
auto schema = adapter->schema();
7476
ICEBERG_ASSIGN_OR_RAISE(
7577
auto writer,
76-
OpenFileWriter(manifest_location, adapter->schema(), std::move(file_io)));
78+
OpenFileWriter(manifest_location, std::move(schema), std::move(file_io)));
7779
return std::make_unique<ManifestWriter>(std::move(writer), std::move(adapter));
7880
}
7981

@@ -84,9 +86,11 @@ Result<std::unique_ptr<ManifestWriter>> ManifestWriter::MakeV2Writer(
8486
std::make_unique<ManifestEntryAdapterV2>(snapshot_id, std::move(partition_spec));
8587
ICEBERG_RETURN_UNEXPECTED(adapter->Init());
8688
ICEBERG_RETURN_UNEXPECTED(adapter->StartAppending());
89+
90+
auto schema = adapter->schema();
8791
ICEBERG_ASSIGN_OR_RAISE(
8892
auto writer,
89-
OpenFileWriter(manifest_location, adapter->schema(), std::move(file_io)));
93+
OpenFileWriter(manifest_location, std::move(schema), std::move(file_io)));
9094
return std::make_unique<ManifestWriter>(std::move(writer), std::move(adapter));
9195
}
9296

@@ -98,9 +102,11 @@ Result<std::unique_ptr<ManifestWriter>> ManifestWriter::MakeV3Writer(
98102
std::move(partition_spec));
99103
ICEBERG_RETURN_UNEXPECTED(adapter->Init());
100104
ICEBERG_RETURN_UNEXPECTED(adapter->StartAppending());
105+
106+
auto schema = adapter->schema();
101107
ICEBERG_ASSIGN_OR_RAISE(
102108
auto writer,
103-
OpenFileWriter(manifest_location, adapter->schema(), std::move(file_io)));
109+
OpenFileWriter(manifest_location, std::move(schema), std::move(file_io)));
104110
return std::make_unique<ManifestWriter>(std::move(writer), std::move(adapter));
105111
}
106112

@@ -134,9 +140,11 @@ Result<std::unique_ptr<ManifestListWriter>> ManifestListWriter::MakeV1Writer(
134140
auto adapter = std::make_unique<ManifestFileAdapterV1>(snapshot_id, parent_snapshot_id);
135141
ICEBERG_RETURN_UNEXPECTED(adapter->Init());
136142
ICEBERG_RETURN_UNEXPECTED(adapter->StartAppending());
143+
144+
auto schema = adapter->schema();
137145
ICEBERG_ASSIGN_OR_RAISE(
138146
auto writer,
139-
OpenFileWriter(manifest_list_location, adapter->schema(), std::move(file_io)));
147+
OpenFileWriter(manifest_list_location, std::move(schema), std::move(file_io)));
140148
return std::make_unique<ManifestListWriter>(std::move(writer), std::move(adapter));
141149
}
142150

@@ -148,9 +156,11 @@ Result<std::unique_ptr<ManifestListWriter>> ManifestListWriter::MakeV2Writer(
148156
sequence_number);
149157
ICEBERG_RETURN_UNEXPECTED(adapter->Init());
150158
ICEBERG_RETURN_UNEXPECTED(adapter->StartAppending());
159+
160+
auto schema = adapter->schema();
151161
ICEBERG_ASSIGN_OR_RAISE(
152162
auto writer,
153-
OpenFileWriter(manifest_list_location, adapter->schema(), std::move(file_io)));
163+
OpenFileWriter(manifest_list_location, std::move(schema), std::move(file_io)));
154164

155165
return std::make_unique<ManifestListWriter>(std::move(writer), std::move(adapter));
156166
}
@@ -163,9 +173,11 @@ Result<std::unique_ptr<ManifestListWriter>> ManifestListWriter::MakeV3Writer(
163173
sequence_number, first_row_id);
164174
ICEBERG_RETURN_UNEXPECTED(adapter->Init());
165175
ICEBERG_RETURN_UNEXPECTED(adapter->StartAppending());
176+
177+
auto schema = adapter->schema();
166178
ICEBERG_ASSIGN_OR_RAISE(
167179
auto writer,
168-
OpenFileWriter(manifest_list_location, adapter->schema(), std::move(file_io)));
180+
OpenFileWriter(manifest_list_location, std::move(schema), std::move(file_io)));
169181
return std::make_unique<ManifestListWriter>(std::move(writer), std::move(adapter));
170182
}
171183

0 commit comments

Comments
 (0)