Skip to content

Increase expiry threshold in LruCacheEviction test - #1729

Closed
rustam-gamidov-here wants to merge 1 commit into
masterfrom
rga/lru_eviction_timeout
Closed

rustam-gamidov-here wants to merge 1 commit into
masterfrom
rga/lru_eviction_timeout

Conversation

@rustam-gamidov-here

Copy link
Copy Markdown
Collaborator

In shared containerized environment timeouts may be not stable. Locally it can be reproduced running heavy build in parallel. There's no need to keep this timeout too small as it should just keep the key alive.

Relates-To: DATASDK-110

@codecov

codecov Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.45%. Comparing base (0c2c25a) to head (74c39c8).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1729      +/-   ##
==========================================
- Coverage   80.45%   80.45%   -0.01%     
==========================================
  Files         358      358              
  Lines       14443    14444       +1     
  Branches     1583     1581       -2     
==========================================
  Hits        11620    11620              
+ Misses       2182     2180       -2     
- Partials      641      644       +3     
Files with missing lines Coverage Δ
...dk-authentication/src/AuthenticationClientImpl.cpp 81.75% <100.00%> (ø)
...k-authentication/src/AuthenticationClientUtils.cpp 91.03% <100.00%> (ø)
...p-cpp-sdk-authentication/src/TokenEndpointImpl.cpp 84.09% <100.00%> (ø)
...p-cpp-sdk-core/include/olp/core/porting/optional.h 100.00% <ø> (ø)
...dataservice-read/src/generated/parser/JsonParser.h 91.30% <100.00%> (ø)
...ice-read/src/repositories/PartitionsSaxHandler.cpp 95.56% <100.00%> (ø)
...rvice-read/src/repositories/PartitionsSaxHandler.h 70.83% <ø> (ø)
...ataservice-read/src/repositories/QuadTreeIndex.cpp 84.44% <100.00%> (ø)
...ataservice-write/src/generated/parser/JsonParser.h 85.71% <100.00%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rustam-gamidov-here
rustam-gamidov-here force-pushed the rga/lru_eviction_timeout branch 5 times, most recently from ec30623 to 0b1cc67 Compare October 21, 2026 13:45
In shared containerized environment timeouts may be not stable.
Locally it can be reproduced running heavy build in parallel.
There's no need to keep this timeout too small as it should just keep
the key alive.

Relates-To: DATASDK-110

Signed-off-by: Rustam Gamidov <ext-rustam.gamidov@here.com>
@rustam-gamidov-here
rustam-gamidov-here deleted the rga/lru_eviction_timeout branch October 21, 2026 16:59
@rustam-gamidov-here
rustam-gamidov-here restored the rga/lru_eviction_timeout branch October 22, 2026 05:36
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.

1 participant