qidaye commented on code in PR #28810: URL: https://github.com/apache/doris/pull/28810#discussion_r1436756509
########## be/src/olap/rowset/segment_v2/inverted_index_compound_directory.cpp: ########## @@ -534,17 +532,12 @@ void DorisCompoundDirectory::init(const io::FileSystemSPtr& _fs, const char* _pa } else { cfs_directory = _path; } - bool doClearLockID = false; if (lock_factory == nullptr) { lock_factory = _CLNEW lucene::store::NoLockFactory(); } - setLockFactory(lock_factory); - - if (doClearLockID) { Review Comment: `doClearLockID` is always false in this context and the if condition is never reached, so I removed it. -- 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