airborne12 commented on code in PR #35473:
URL: https://github.com/apache/doris/pull/35473#discussion_r1617054788
##########
be/src/olap/rowset/segment_v2/inverted_index_file_writer.cpp:
##########
@@ -41,33 +41,32 @@ Status
InvertedIndexFileWriter::initialize(InvertedIndexDirectoryMap& indices_di
}
Result<DorisFSDirectory*> InvertedIndexFileWriter::open(const TabletIndex*
index_meta) {
- auto index_id = index_meta->index_id();
- auto index_suffix = index_meta->get_index_suffix();
auto tmp_file_dir =
ExecEnv::GetInstance()->get_tmp_file_dirs()->get_tmp_file_dir();
- _lfs = io::global_local_filesystem();
- auto lfs_index_path = InvertedIndexDescriptor::get_temporary_index_path(
- tmp_file_dir / _segment_file_name, index_meta->index_id(),
+ const auto& local_fs = io::global_local_filesystem();
+ auto local_fs_index_path =
InvertedIndexDescriptor::get_temporary_index_path(
+ tmp_file_dir.native(), _rowset_id, _seg_id, index_meta->index_id(),
index_meta->get_index_suffix());
- auto index_path = InvertedIndexDescriptor::get_temporary_index_path(
- (_index_file_dir / _segment_file_name).native(), index_id,
index_suffix);
+ auto index_path = InvertedIndexDescriptor::get_index_path_v1(
Review Comment:
we need tmp dir here, get_index_path_v1 will give a full path with idx
suffix.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
