Fix session key resources closed unexpectedly #5300
Conversation
…ail consistently.
…s from iterators.
…to make sure contents do not get lost.
tnleeuw
force-pushed
the
issue/5287_ClosedSessionKeys
branch
from
1da27aa to
1404eb3
Compare
|
It looks good, but I don't have all the knowledge yet to approve the whole change (yet). |
| } else if (obj instanceof Message) { | ||
| return ((Message) obj).asString(); |
There was a problem hiding this comment.
Voorheen werd hij gesloten, is dit niet meer de bedoeling? Misschien een commentaartje toevoegen?
There was a problem hiding this comment.
Iets wat al als Message in de PipeLineSession zit, moet niet worden gesloten want wordt misschien opnieuw gelezen.
Ik zal er een comment bij toevoegen!
| } | ||
|
|
||
| @Test | ||
| public void testCopyMessage3() throws IOException { |
There was a problem hiding this comment.
Ik geloof wel dat dit goed gaat, maar eigenlijk zouden we dit ook met een grote file moeten testen.
There was a problem hiding this comment.
Ik geloof wel dat dit goed gaat, maar eigenlijk zouden we dit ook met een grote file moeten testen.
Iedere stream van onbekende size wordt als SerializableFileReference ge-preserved dus zelfde als een grote file, volgens mij?
Daarom heb ik niet geprobeerd om dat apart te testen.
|
Kudos, SonarCloud Quality Gate passed!
|
|
Please retry analysis of this Pull-Request directly on SonarCloud. |
No description provided.