Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

BufferTimeout or FluxFlatmapIterable fails to discard all elements #3531

Open
OlegDokuka opened this issue Jul 5, 2023 · 0 comments · May be fixed by #3634
Open

BufferTimeout or FluxFlatmapIterable fails to discard all elements #3531

OlegDokuka opened this issue Jul 5, 2023 · 0 comments · May be fixed by #3634
Assignees
@chemicL
Labels
status/need-investigation This needs more in-depth investigation type/bug A general bug
Milestone
3.5.17

Comments

@OlegDokuka
Copy link
Contributor

OlegDokuka commented Jul 5, 2023 •
edited 

OnDiscardShouldNotLeakTest > Populated queue racing Cancel/overflow Error > Populated queue racing Cancel/overflow Error [73] for bufferTimeout FAILED
    java.lang.AssertionError: 
    Expecting all elements of:
      [Tracked{ id=0 released=true },
        Tracked{ id=1 released=true },
        Tracked{ id=2 released=true },
        Tracked{ id=3 released=true },
        Tracked{ id=4 released=true },
        Tracked{ id=5 released=true },
        Tracked{ id=6 released=true },
        Tracked{ id=7 released=true },
        Tracked{ id=8 released=true },
        Tracked{ id=9 released=true },
        Tracked{ id=10 released=true },
        Tracked{ id=11 released=true },
        Tracked{ id=12 released=true },
        Tracked{ id=13 released=true },
        Tracked{ id=14 released=true },
        Tracked{ id=15 released=true },
        Tracked{ id=16 released=true },
        Tracked{ id=17 released=true },
        Tracked{ id=18 released=true },
        Tracked{ id=19 released=true },
        Tracked{ id=20 released=true },
        Tracked{ id=21 released=true },
        Tracked{ id=22 released=true },
        Tracked{ id=23 released=true },
        Tracked{ id=24 released=true },
        Tracked{ id=25 released=true },
        Tracked{ id=26 released=true },
        Tracked{ id=27 released=true },
        Tracked{ id=28 released=true },
        Tracked{ id=29 released=true },
        Tracked{ id=30 released=true },
        Tracked{ id=31 released=true },
        Tracked{ id=32 released=true },
        Tracked{ id=33 released=true },
        Tracked{ id=34 released=true },
        Tracked{ id=35 released=true },
        Tracked{ id=36 released=true },
        Tracked{ id=37 released=true },
        Tracked{ id=38 released=true },
        Tracked{ id=39 released=true },
        Tracked{ id=40 released=true },
        Tracked{ id=41 released=true },
        Tracked{ id=42 released=true },
        Tracked{ id=43 released=true },
        Tracked{ id=44 released=true },
        Tracked{ id=45 released=true },
        Tracked{ id=46 released=true },
        Tracked{ id=47 released=true },
        Tracked{ id=48 released=true },
        Tracked{ id=49 released=true },
        Tracked{ id=50 released=true },
        Tracked{ id=51 released=true },
        Tracked{ id=52 released=true },
        Tracked{ id=53 released=true },
        Tracked{ id=54 released=true },
        Tracked{ id=55 released=true },
        Tracked{ id=56 released=true },
        Tracked{ id=57 released=true },
        Tracked{ id=58 released=true },
        Tracked{ id=59 released=true },
        Tracked{ id=60 released=true },
        Tracked{ id=61 released=true },
        Tracked{ id=62 released=true },
        Tracked{ id=63 released=true },
        Tracked{ id=64 released=true },
        Tracked{ id=65 released=true },
        Tracked{ id=66 released=true },
        Tracked{ id=67 released=true },
        Tracked{ id=68 released=true },
        Tracked{ id=69 released=true },
        Tracked{ id=70 released=true },
        Tracked{ id=71 released=true },
        Tracked{ id=72 released=true },
        Tracked{ id=73 released=true },
        Tracked{ id=74 released=true },
        Tracked{ id=75 released=true },
        Tracked{ id=76 released=true },
        Tracked{ id=77 released=true },
        Tracked{ id=78 released=true },
        Tracked{ id=79 released=true },
        Tracked{ id=80 released=true },
        Tracked{ id=81 released=true },
        Tracked{ id=82 released=true },
        Tracked{ id=83 released=true },
        Tracked{ id=84 released=true },
        Tracked{ id=85 released=true },
        Tracked{ id=86 released=true },
        Tracked{ id=87 released=true },
        Tracked{ id=88 released=true },
        Tracked{ id=89 released=true },
        Tracked{ id=90 released=true },
        Tracked{ id=91 released=true },
        Tracked{ id=92 released=true },
        Tracked{ id=93 released=true },
        Tracked{ id=94 released=true },
        Tracked{ id=95 released=true },
        Tracked{ id=96 released=true },
        Tracked{ id=97 released=true },
        Tracked{ id=98 released=true },
        Tracked{ id=99 released=true },
        Tracked{ id=100 released=true },
        Tracked{ id=101 released=true },
        Tracked{ id=102 released=true },
        Tracked{ id=103 released=true },
        Tracked{ id=104 released=true },
        Tracked{ id=105 released=true },
        Tracked{ id=106 released=true },
        Tracked{ id=107 released=true },
        Tracked{ id=108 released=true },
        Tracked{ id=109 released=true },
        Tracked{ id=110 released=true },
        Tracked{ id=111 released=true },
        Tracked{ id=112 released=true },
        Tracked{ id=113 released=true },
        Tracked{ id=114 released=true },
        Tracked{ id=115 released=true },
        Tracked{ id=116 released=true },
        Tracked{ id=117 released=true },
        Tracked{ id=118 released=true },
        Tracked{ id=119 released=true },
        Tracked{ id=120 released=true },
        Tracked{ id=121 released=true },
        Tracked{ id=122 released=true },
        Tracked{ id=123 released=true },
        Tracked{ id=124 released=true },
        Tracked{ id=125 released=true },
        Tracked{ id=126 released=true },
        Tracked{ id=127 released=true },
        Tracked{ id=128 released=true },
        Tracked{ id=129 released=true },
        Tracked{ id=130 released=true },
        Tracked{ id=131 released=true },
        Tracked{ id=132 released=true },
        Tracked{ id=133 released=true },
        Tracked{ id=134 released=true },
        Tracked{ id=135 released=true },
        Tracked{ id=136 released=true },
        Tracked{ id=137 released=true },
        Tracked{ id=138 released=true },
        Tracked{ id=139 released=true },
        Tracked{ id=140 released=true },
        Tracked{ id=141 released=true },
        Tracked{ id=142 released=true },
        Tracked{ id=143 released=true },
        Tracked{ id=144 released=true },
        Tracked{ id=145 released=true },
        Tracked{ id=146 released=true },
        Tracked{ id=147 released=true },
        Tracked{ id=148 released=true },
        Tracked{ id=149 released=true },
        Tracked{ id=150 released=true },
        Tracked{ id=151 released=true },
        Tracked{ id=152 released=true },
        Tracked{ id=153 released=true },
        Tracked{ id=154 released=true },
        Tracked{ id=155 released=true },
        Tracked{ id=156 released=true },
        Tracked{ id=157 released=true },
        Tracked{ id=158 released=true },
        Tracked{ id=159 released=true },
        Tracked{ id=160 released=true },
        Tracked{ id=161 released=true },
        Tracked{ id=162 released=true },
        Tracked{ id=163 released=true },
        Tracked{ id=164 released=true },
        Tracked{ id=165 released=true },
        Tracked{ id=166 released=true },
        Tracked{ id=167 released=true },
        Tracked{ id=168 released=true },
        Tracked{ id=169 released=true },
        Tracked{ id=170 released=true },
        Tracked{ id=171 released=true },
        Tracked{ id=172 released=true },
        Tracked{ id=173 released=true },
        Tracked{ id=174 released=true },
        Tracked{ id=175 released=true },
        Tracked{ id=176 released=true },
        Tracked{ id=177 released=true },
        Tracked{ id=178 released=true },
        Tracked{ id=179 released=true },
        Tracked{ id=180 released=true },
        Tracked{ id=181 released=true },
        Tracked{ id=182 released=true },
        Tracked{ id=183 released=true },
        Tracked{ id=184 released=true },
        Tracked{ id=185 released=true },
        Tracked{ id=186 released=true },
        Tracked{ id=187 released=true },
        Tracked{ id=188 released=true },
        Tracked{ id=189 released=true },
        Tracked{ id=190 released=true },
        Tracked{ id=191 released=true },
        Tracked{ id=192 released=true },
        Tracked{ id=193 released=true },
        Tracked{ id=194 released=true },
        Tracked{ id=195 released=true },
        Tracked{ id=196 released=true },
        Tracked{ id=197 released=true },
        Tracked{ id=198 released=true },
        Tracked{ id=199 released=true },
        Tracked{ id=200 released=true },
        Tracked{ id=201 released=true },
        Tracked{ id=202 released=true },
        Tracked{ id=203 released=true },
        Tracked{ id=204 released=true },
        Tracked{ id=205 released=true },
        Tracked{ id=206 released=true },
        Tracked{ id=207 released=true },
        Tracked{ id=208 released=true },
        Tracked{ id=209 released=true },
        Tracked{ id=210 released=true },
        Tracked{ id=211 released=true },
        Tracked{ id=212 released=true },
        Tracked{ id=213 released=true },
        Tracked{ id=214 released=true },
        Tracked{ id=215 released=true },
        Tracked{ id=216 released=true },
        Tracked{ id=217 released=true },
        Tracked{ id=218 released=true },
        Tracked{ id=219 released=true },
        Tracked{ id=220 released=true },
        Tracked{ id=221 released=true },
        Tracked{ id=222 released=true },
        Tracked{ id=223 released=true },
        Tracked{ id=224 released=true },
        Tracked{ id=225 released=true },
        Tracked{ id=226 released=true },
        Tracked{ id=227 released=true },
        Tracked{ id=228 released=true },
        Tracked{ id=229 released=true },
        Tracked{ id=230 released=true },
        Tracked{ id=231 released=true },
        Tracked{ id=232 released=true },
        Tracked{ id=233 released=true },
        Tracked{ id=234 released=true },
        Tracked{ id=235 released=true },
        Tracked{ id=236 released=true },
        Tracked{ id=237 released=true },
        Tracked{ id=238 released=true },
        Tracked{ id=239 released=true },
        Tracked{ id=240 released=true },
        Tracked{ id=241 released=true },
        Tracked{ id=242 released=true },
        Tracked{ id=243 released=true },
        Tracked{ id=244 released=true },
        Tracked{ id=245 released=true },
        Tracked{ id=246 released=true },
        Tracked{ id=247 released=true },
        Tracked{ id=248 released=true },
        Tracked{ id=249 released=true },
        Tracked{ id=250 released=true },
        Tracked{ id=251 released=true },
        Tracked{ id=252 released=true },
        Tracked{ id=253 released=true },
        Tracked{ id=254 released=true },
        Tracked{ id=255 released=false },
        Tracked{ id=256 released=true },
        Tracked{ id=257 released=true }]
    to match 'isReleased' predicate but this element did not:
      Tracked{ id=255 released=false }
        at reactor.test.MemoryUtils$OffHeapDetector.assertNoLeaks(MemoryUtils.java:144)
        at reactor.core.publisher.OnDiscardShouldNotLeakTest.ensureNoLeaksPopulatedQueueAndRacingCancelAndOverflowError(OnDiscardShouldNotLeakTest.java:586)

OnDiscardShouldNotLeakTest > Populated queue racing Cancel/overflow Error > Populated queue racing Cancel/overflow Error [75] for bufferTimeout (fused) FAILED
    java.lang.AssertionError: 
    Expecting all elements of:
      [Tracked{ id=0 released=true },
        Tracked{ id=1 released=true },
        Tracked{ id=2 released=true },
        Tracked{ id=3 released=true },
        Tracked{ id=4 released=true },
        Tracked{ id=5 released=true },
        Tracked{ id=6 released=true },
        Tracked{ id=7 released=true },
        Tracked{ id=8 released=true },
        Tracked{ id=9 released=true },
        Tracked{ id=10 released=true },
        Tracked{ id=11 released=true },
        Tracked{ id=12 released=true },
        Tracked{ id=13 released=true },
        Tracked{ id=14 released=true },
        Tracked{ id=15 released=true },
        Tracked{ id=16 released=true },
        Tracked{ id=17 released=true },
        Tracked{ id=18 released=true },
        Tracked{ id=19 released=true },
        Tracked{ id=20 released=true },
        Tracked{ id=21 released=true },
        Tracked{ id=22 released=true },
        Tracked{ id=23 released=true },
        Tracked{ id=24 released=true },
        Tracked{ id=25 released=true },
        Tracked{ id=26 released=true },
        Tracked{ id=27 released=true },
        Tracked{ id=28 released=true },
        Tracked{ id=29 released=true },
        Tracked{ id=30 released=true },
        Tracked{ id=31 released=true },
        Tracked{ id=32 released=true },
        Tracked{ id=33 released=true },
        Tracked{ id=34 released=true },
        Tracked{ id=35 released=true },
        Tracked{ id=36 released=true },
        Tracked{ id=37 released=true },
        Tracked{ id=38 released=true },
        Tracked{ id=39 released=true },
        Tracked{ id=40 released=true },
        Tracked{ id=41 released=true },
        Tracked{ id=42 released=true },
        Tracked{ id=43 released=true },
        Tracked{ id=44 released=true },
        Tracked{ id=45 released=true },
        Tracked{ id=46 released=true },
        Tracked{ id=47 released=true },
        Tracked{ id=48 released=true },
        Tracked{ id=49 released=true },
        Tracked{ id=50 released=true },
        Tracked{ id=51 released=true },
        Tracked{ id=52 released=true },
        Tracked{ id=53 released=true },
        Tracked{ id=54 released=true },
        Tracked{ id=55 released=true },
        Tracked{ id=56 released=true },
        Tracked{ id=57 released=true },
        Tracked{ id=58 released=true },
        Tracked{ id=59 released=true },
        Tracked{ id=60 released=true },
        Tracked{ id=61 released=true },
        Tracked{ id=62 released=true },
        Tracked{ id=63 released=true },
        Tracked{ id=64 released=true },
        Tracked{ id=65 released=true },
        Tracked{ id=66 released=true },
        Tracked{ id=67 released=true },
        Tracked{ id=68 released=true },
        Tracked{ id=69 released=true },
        Tracked{ id=70 released=true },
        Tracked{ id=71 released=true },
        Tracked{ id=72 released=true },
        Tracked{ id=73 released=true },
        Tracked{ id=74 released=true },
        Tracked{ id=75 released=true },
        Tracked{ id=76 released=true },
        Tracked{ id=77 released=true },
        Tracked{ id=78 released=true },
        Tracked{ id=79 released=true },
        Tracked{ id=80 released=true },
        Tracked{ id=81 released=true },
        Tracked{ id=82 released=true },
        Tracked{ id=83 released=true },
        Tracked{ id=84 released=true },
        Tracked{ id=85 released=true },
        Tracked{ id=86 released=true },
        Tracked{ id=87 released=true },
        Tracked{ id=88 released=true },
        Tracked{ id=89 released=true },
        Tracked{ id=90 released=true },
        Tracked{ id=91 released=true },
        Tracked{ id=92 released=true },
        Tracked{ id=93 released=true },
        Tracked{ id=94 released=true },
        Tracked{ id=95 released=true },
        Tracked{ id=96 released=true },
        Tracked{ id=97 released=true },
        Tracked{ id=98 released=true },
        Tracked{ id=99 released=true },
        Tracked{ id=100 released=true },
        Tracked{ id=101 released=true },
        Tracked{ id=102 released=true },
        Tracked{ id=103 released=true },
        Tracked{ id=104 released=true },
        Tracked{ id=105 released=true },
        Tracked{ id=106 released=true },
        Tracked{ id=107 released=true },
        Tracked{ id=108 released=true },
        Tracked{ id=109 released=true },
        Tracked{ id=110 released=true },
        Tracked{ id=111 released=true },
        Tracked{ id=112 released=true },
        Tracked{ id=113 released=true },
        Tracked{ id=114 released=true },
        Tracked{ id=115 released=true },
        Tracked{ id=116 released=true },
        Tracked{ id=117 released=true },
        Tracked{ id=118 released=true },
        Tracked{ id=119 released=true },
        Tracked{ id=120 released=true },
        Tracked{ id=121 released=true },
        Tracked{ id=122 released=true },
        Tracked{ id=123 released=true },
        Tracked{ id=124 released=true },
        Tracked{ id=125 released=true },
        Tracked{ id=126 released=true },
        Tracked{ id=127 released=true },
        Tracked{ id=128 released=true },
        Tracked{ id=129 released=true },
        Tracked{ id=130 released=true },
        Tracked{ id=131 released=true },
        Tracked{ id=132 released=true },
        Tracked{ id=133 released=true },
        Tracked{ id=134 released=true },
        Tracked{ id=135 released=true },
        Tracked{ id=136 released=true },
        Tracked{ id=137 released=true },
        Tracked{ id=138 released=true },
        Tracked{ id=139 released=true },
        Tracked{ id=140 released=true },
        Tracked{ id=141 released=true },
        Tracked{ id=142 released=true },
        Tracked{ id=143 released=true },
        Tracked{ id=144 released=true },
        Tracked{ id=145 released=true },
        Tracked{ id=146 released=true },
        Tracked{ id=147 released=true },
        Tracked{ id=148 released=true },
        Tracked{ id=149 released=true },
        Tracked{ id=150 released=true },
        Tracked{ id=151 released=true },
        Tracked{ id=152 released=true },
        Tracked{ id=153 released=true },
        Tracked{ id=154 released=true },
        Tracked{ id=155 released=true },
        Tracked{ id=156 released=true },
        Tracked{ id=157 released=true },
        Tracked{ id=158 released=true },
        Tracked{ id=159 released=true },
        Tracked{ id=160 released=true },
        Tracked{ id=161 released=true },
        Tracked{ id=162 released=true },
        Tracked{ id=163 released=true },
        Tracked{ id=164 released=true },
        Tracked{ id=165 released=true },
        Tracked{ id=166 released=true },
        Tracked{ id=167 released=true },
        Tracked{ id=168 released=true },
        Tracked{ id=169 released=true },
        Tracked{ id=170 released=true },
        Tracked{ id=171 released=true },
        Tracked{ id=172 released=true },
        Tracked{ id=173 released=true },
        Tracked{ id=174 released=true },
        Tracked{ id=175 released=true },
        Tracked{ id=176 released=true },
        Tracked{ id=177 released=true },
        Tracked{ id=178 released=true },
        Tracked{ id=179 released=true },
        Tracked{ id=180 released=true },
        Tracked{ id=181 released=true },
        Tracked{ id=182 released=true },
        Tracked{ id=183 released=true },
        Tracked{ id=184 released=true },
        Tracked{ id=185 released=true },
        Tracked{ id=186 released=true },
        Tracked{ id=187 released=true },
        Tracked{ id=188 released=true },
        Tracked{ id=189 released=true },
        Tracked{ id=190 released=true },
        Tracked{ id=191 released=true },
        Tracked{ id=192 released=true },
        Tracked{ id=193 released=true },
        Tracked{ id=194 released=true },
        Tracked{ id=195 released=true },
        Tracked{ id=196 released=true },
        Tracked{ id=197 released=true },
        Tracked{ id=198 released=true },
        Tracked{ id=199 released=true },
        Tracked{ id=200 released=true },
        Tracked{ id=201 released=true },
        Tracked{ id=202 released=true },
        Tracked{ id=203 released=true },
        Tracked{ id=204 released=true },
        Tracked{ id=205 released=true },
        Tracked{ id=206 released=true },
        Tracked{ id=207 released=true },
        Tracked{ id=208 released=true },
        Tracked{ id=209 released=true },
        Tracked{ id=210 released=true },
        Tracked{ id=211 released=true },
        Tracked{ id=212 released=true },
        Tracked{ id=213 released=true },
        Tracked{ id=214 released=true },
        Tracked{ id=215 released=true },
        Tracked{ id=216 released=true },
        Tracked{ id=217 released=true },
        Tracked{ id=218 released=true },
        Tracked{ id=219 released=true },
        Tracked{ id=220 released=true },
        Tracked{ id=221 released=true },
        Tracked{ id=222 released=true },
        Tracked{ id=223 released=true },
        Tracked{ id=224 released=true },
        Tracked{ id=225 released=true },
        Tracked{ id=226 released=true },
        Tracked{ id=227 released=true },
        Tracked{ id=228 released=true },
        Tracked{ id=229 released=true },
        Tracked{ id=230 released=true },
        Tracked{ id=231 released=true },
        Tracked{ id=232 released=true },
        Tracked{ id=233 released=true },
        Tracked{ id=234 released=true },
        Tracked{ id=235 released=true },
        Tracked{ id=236 released=true },
        Tracked{ id=237 released=true },
        Tracked{ id=238 released=true },
        Tracked{ id=239 released=true },
        Tracked{ id=240 released=true },
        Tracked{ id=241 released=true },
        Tracked{ id=242 released=true },
        Tracked{ id=243 released=true },
        Tracked{ id=244 released=true },
        Tracked{ id=245 released=true },
        Tracked{ id=246 released=true },
        Tracked{ id=247 released=true },
        Tracked{ id=248 released=true },
        Tracked{ id=249 released=true },
        Tracked{ id=250 released=true },
        Tracked{ id=251 released=true },
        Tracked{ id=252 released=true },
        Tracked{ id=253 released=true },
        Tracked{ id=254 released=true },
        Tracked{ id=255 released=false },
        Tracked{ id=256 released=true },
        Tracked{ id=257 released=true }]
    to match 'isReleased' predicate but this element did not:
      Tracked{ id=255 released=false }
        at reactor.test.MemoryUtils$OffHeapDetector.assertNoLeaks(MemoryUtils.java:144)
        at reactor.core.publisher.OnDiscardShouldNotLeakTest.ensureNoLeaksPopulatedQueueAndRacingCancelAndOverflowError(OnDiscardShouldNotLeakTest.java:586)
@reactorbot reactorbot added the ❓need-triage This issue needs triage, hasn't been looked at by a team member yet label Jul 5, 2023
@OlegDokuka OlegDokuka added type/bug A general bug status/need-investigation This needs more in-depth investigation and removed ❓need-triage This issue needs triage, hasn't been looked at by a team member yet labels Jul 5, 2023
@OlegDokuka OlegDokuka changed the title investigate flaky onDiscard test BufferTimeout or FluxFlatmapIterable fails to discard all elements Jul 7, 2023
@chemicL chemicL linked a pull request Nov 7, 2023 that will close this issue
Draft
@chemicL chemicL mentioned this issue Mar 20, 2024
Merged
@chemicL chemicL modified the milestones: 3.5.x Backlog, 3.5.16 Mar 20, 2024
@chemicL chemicL linked a pull request Mar 20, 2024 that will close this issue
BufferTimeout with fair backpressure rework #3634
Draft
chemicL added a commit that referenced this issue Mar 20, 2024
@chemicL
[test] Ignore bufferTimeout discard validation (#3767)
70339f5 
Until #3531 is resolved the bufferTimeout discard support validation can
be ignored to avoid build failures.
@chemicL chemicL modified the milestones: 3.5.16, 3.5.17 Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chemicL chemicL

Labels
status/need-investigation This needs more in-depth investigation type/bug A general bug
Projects
None yet
Milestone
3.5.17
Development

Successfully merging a pull request may close this issue.

BufferTimeout with fair backpressure rework reactor/reactor-core
3 participants
@chemicL @OlegDokuka @reactorbot