Skip to content

Commit

Permalink
[fix](load) delete bytes_appended debug point which may cause bad page (
Browse files Browse the repository at this point in the history
  • Loading branch information
liaoxin01 authored Feb 29, 2024
1 parent 31ad7a2 commit 95fc7a2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
5 changes: 1 addition & 4 deletions be/src/io/fs/file_writer.h
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,7 @@ class FileWriter {

const Path& path() const { return _path; }

size_t bytes_appended() const {
DBUG_EXECUTE_IF("FileWriter.bytes_appended.zero_bytes_appended", { return 0; });
return _bytes_appended;
}
size_t bytes_appended() const { return _bytes_appended; }

std::shared_ptr<FileSystem> fs() const { return _fs; }

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -143,8 +143,6 @@ suite("load_stream_fault_injection", "nonConcurrent") {
load_with_injection("LocalFileSystem.create_file_impl.open_file_failed", "")
// LoadStreamWriter append_data meet null file writer error
load_with_injection("LoadStreamWriter.append_data.null_file_writer", "")
// LoadStreamWriter append_data meet bytes_appended and real file size not match error
load_with_injection("FileWriter.bytes_appended.zero_bytes_appended", "")
// LoadStreamWriter close_segment meet not inited error
load_with_injection("LoadStreamWriter.close_segment.uninited_writer", "")
// LoadStreamWriter close_segment meet not bad segid error
Expand Down

0 comments on commit 95fc7a2

Please sign in to comment.