diff --git a/be/src/olap/rowset/segment_v2/column_writer.cpp b/be/src/olap/rowset/segment_v2/column_writer.cpp index b203a39997a5bf..5b7eac683548f9 100644 --- a/be/src/olap/rowset/segment_v2/column_writer.cpp +++ b/be/src/olap/rowset/segment_v2/column_writer.cpp @@ -727,12 +727,12 @@ Status ScalarColumnWriter::finish_current_page() { if (has_null) { OwnedSlice tmp_map(std::move(body.back())); body.pop_back(); - OwnedSlice tmp_value(std::move(body.back()));; + OwnedSlice tmp_value(std::move(body.back())); body.pop_back(); page->data.emplace_back(std::move(tmp_value)); page->data.emplace_back(std::move(tmp_map)); } else { - OwnedSlice tmp_value(std::move(body.back()));; + OwnedSlice tmp_value(std::move(body.back())); body.pop_back(); page->data.emplace_back(std::move(tmp_value)); } diff --git a/be/src/olap/rowset/segment_v2/page_io.cpp b/be/src/olap/rowset/segment_v2/page_io.cpp index 9eadcacaa84e2e..e9adb58469a16b 100644 --- a/be/src/olap/rowset/segment_v2/page_io.cpp +++ b/be/src/olap/rowset/segment_v2/page_io.cpp @@ -95,15 +95,17 @@ Status PageIO::write_page(io::FileWriter* writer, std::vector& body, put_fixed32_le(&footer_buf, static_cast(footer_buf.size())); std::vector& page = body; - faststring footer_and_checksum; - footer_and_checksum.assign_copy(footer_buf); + faststring footer_str; + footer_str.assign_copy(footer_buf); + page.emplace_back(footer_str.build()); // checksum uint8_t checksum_buf[sizeof(uint32_t)]; uint32_t checksum = crc32c::Value(page); encode_fixed32_le(checksum_buf, checksum); - footer_and_checksum.append(checksum_buf, sizeof(uint32_t)); - page.emplace_back(footer_and_checksum.build()); + faststring checksum_str; + checksum_str.append(checksum_buf, sizeof(uint32_t)); + page.emplace_back(checksum_str.build()); uint64_t offset = writer->bytes_appended(); RETURN_IF_ERROR(writer->appendv(&page[0], page.size()));