Re: Re[4]: IGNITE-6499 Compact NULL fields

2020-06-12 Thread steve.hostett...@gmail.com
Hello, Stanilovsky Evgeny : would you mind having a look at the pull request. Thanks in advance -- Sent from: http://apache-ignite-developers.2346864.n4.nabble.com/

Re: Re[4]: IGNITE-6499 Compact NULL fields

2020-05-28 Thread steve.hostett...@gmail.com
Hello, still fighting to get my unit tests green. Something I do not really understand is the mechanism of object replacement. For instance, marshalling a LocalDateTime, gets replace by java.time.Ser, I assume that's because it has a method writeReplace. But that takes precedence over a custom ser

Re: Re[4]: IGNITE-6499 Compact NULL fields

2020-05-25 Thread steve.hostett...@gmail.com
Ok, anyway I will finish my patch 2 unit tests still not working. I assume anyway that if we would want to apply zstd compression we would reuse the existing page compression algorithm in memory and not only for persistence. That would probably be anyway simpler and more straightforward. Will try

Re[4]: IGNITE-6499 Compact NULL fields

2020-05-25 Thread Zhenya Stanilovsky
Compress of whole binary inside ignite.   >Sorry I do not actual get what are you opposing? the compress of the binary >or the null compaction or both? >And can you ellaborate on why you are opposing it? > > > >-- >Sent from: http://apache-ignite-developers.2346864.n4.nabble.com/