Skip to content

Conversation

@xinyiZzz
Copy link
Contributor

…er expired (apache#41754)

Guava LoadingCache expiration check is performed when the cache is
accessed, not immediately when the expiration time is reached.

So manually call the cleanUp method regularly to clean up expired cache
items.
@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@xinyiZzz
Copy link
Contributor Author

run buildall

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@xinyiZzz xinyiZzz force-pushed the branch-2.1_20241021_pick_arrow branch from 3922157 to 06024cf Compare October 22, 2024 03:06
@xinyiZzz
Copy link
Contributor Author

run buildall

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

…htSqlConnectProcessor close (apache#41770)

1. Doris will cancel the connection that has not responded for a long
time, Mysql Conenction will exit directly, but Arrow Flight Conenction
does not exit immediately, may be frequently cancel and print logs.
timeout is `wait_timeout` in session veriable.
2. FlightSqlConnectProcessor Use try-with-resources to close correctly.
@xinyiZzz xinyiZzz force-pushed the branch-2.1_20241021_pick_arrow branch from 06024cf to 27d8fbb Compare October 22, 2024 03:21
@xinyiZzz
Copy link
Contributor Author

run buildall

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 36.01% (9347/25958)
Line Coverage: 27.59% (76822/278428)
Region Coverage: 26.40% (39459/149481)
Branch Coverage: 23.18% (20085/86640)
Coverage Report: http://coverage.selectdb-in.cc/coverage/27d8fbbeb619cc46a70714ae0b5c2162b045fefb_27d8fbbeb619cc46a70714ae0b5c2162b045fefb/report/index.html

@yiguolei yiguolei merged commit 6f2bac0 into apache:branch-2.1 Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants