Re: [PR] [improve](cache) File cache async init [doris]

2024-08-15 Thread via GitHub
morningman merged PR #39036: URL: https://github.com/apache/doris/pull/39036 -- 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.a

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-14 Thread via GitHub
github-actions[bot] commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2288166180 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-14 Thread via GitHub
suxiaogang223 commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2288143962 run buildall -- 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. T

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-13 Thread via GitHub
github-actions[bot] commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2286719509 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-13 Thread via GitHub
suxiaogang223 commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2286708583 run buildall -- 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. T

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-09 Thread via GitHub
morningman commented on code in PR #39036: URL: https://github.com/apache/doris/pull/39036#discussion_r1708760493 ## be/src/io/cache/block/block_lru_file_cache.cpp: ## @@ -376,6 +389,16 @@ void LRUFileCache::fill_holes_with_empty_file_blocks(FileBlocks& file_blocks, co FileB

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-09 Thread via GitHub
github-actions[bot] commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2277677877 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-09 Thread via GitHub
suxiaogang223 commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2277576558 run buildall -- 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. T

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-09 Thread via GitHub
github-actions[bot] commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2277547074 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-08 Thread via GitHub
suxiaogang223 commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2276389595 run buildall -- 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. T

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-07 Thread via GitHub
github-actions[bot] commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2275088862 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-07 Thread via GitHub
github-actions[bot] commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2272935359 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-07 Thread via GitHub
github-actions[bot] commented on code in PR #39036: URL: https://github.com/apache/doris/pull/39036#discussion_r1706598421 ## be/src/io/cache/block/block_lru_file_cache.cpp: ## @@ -799,34 +819,26 @@ std::string key_prefix = fs::p

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-07 Thread via GitHub
github-actions[bot] commented on code in PR #39036: URL: https://github.com/apache/doris/pull/39036#discussion_r1706573212 ## be/src/io/cache/block/block_lru_file_cache.cpp: ## @@ -799,34 +815,27 @@ std::string key_prefix = fs::p

[PR] [improve](cache) File cache async init [doris]

2024-08-07 Thread via GitHub
suxiaogang223 opened a new pull request, #39036: URL: https://github.com/apache/doris/pull/39036 ## Proposed changes Do `load_cache_info_into_memory()` asynchronously in a background thread in `LRUFileCache::initialize()`. When the cache is not ready, `LRUFileCache::get_or_set()` w

Re: [PR] [improve](cache) File cache async init [doris]

2024-08-07 Thread via GitHub
doris-robot commented on PR #39036: URL: https://github.com/apache/doris/pull/39036#issuecomment-2272862542 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+y