Commit 212fe03
authored
[Test] Fix SharedBlobCacheServiceTests.testGetMultiThreaded (#112322)
A cacheFileRegion can be concurrently evicted while its being incref'd.
See this comment
https://github.com/elastic/elasticsearch/blob/98fe686da4c5cb82d4b03719977be428dc7934e7/x-pack/plugin/blob-cache/src/main/java/org/elasticsearch/blobcache/shared/SharedBlobCacheService.java#L1812-L1813
The tryRead method also performs null and eviction check for io before
returing true.
https://github.com/elastic/elasticsearch/blob/98fe686da4c5cb82d4b03719977be428dc7934e7/x-pack/plugin/blob-cache/src/main/java/org/elasticsearch/blobcache/shared/SharedBlobCacheService.java#L926-L931
Resolves: #1123141 parent 0489a37 commit 212fe03
2 files changed
+1
-5
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
154 | 154 | | |
155 | 155 | | |
156 | 156 | | |
157 | | - | |
158 | | - | |
159 | | - | |
160 | 157 | | |
161 | 158 | | |
162 | 159 | | |
| |||
Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
444 | 444 | | |
445 | 445 | | |
446 | 446 | | |
447 | | - | |
448 | 447 | | |
449 | 448 | | |
450 | 449 | | |
| |||
494 | 493 | | |
495 | 494 | | |
496 | 495 | | |
| 496 | + | |
497 | 497 | | |
498 | 498 | | |
499 | 499 | | |
500 | 500 | | |
501 | | - | |
502 | 501 | | |
503 | 502 | | |
504 | 503 | | |
| |||
0 commit comments