nizhikov commented on PR #13709: URL: https://github.com/apache/ignite/pull/13709#issuecomment-6287519216
@NSAmelchev thanks, all three applied: 1. `BinaryReaderExImpl` now calls the constructors directly in all seven places, and `BinariesFactory#binaryOffheapObject` is removed together with its implementation (66c0fdff80a). 2. Replaced the per-method `final` with `public final class BinaryReaderExImpl` (a7db8723b9a). 3. The test uses `U.unmarshal(marshaller, in, null)` on an inline `ZipInputStream`; the local helper is gone (9b0256173a5). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
