IMHO, also STREAMER_FLUSH_REQUEST should be added too. Client can flush large batches instead of closing resource. IMHO this is preferable than creating streamer per batch.
пт, 5 мар. 2021 г. в 14:48, Ivan Daschinsky <ivanda...@gmail.com>: > Pavel, thank you for your effort. > > BTW, are you sure that receiver should be a param of > STREAMER_START_REQUEST? > Receiver is mostly internal stuff and I can hardly imagine why > someone would decide to change defaults. > > пт, 5 мар. 2021 г. в 13:01, Pavel Tupitsyn <ptupit...@apache.org>: > >> Igor, >> >> As per our private conversation, I'll try to provide some perf >> measurements >> for those 4 variants, and more detailed descriptions too. >> >> Thanks! >> >> On Fri, Mar 5, 2021 at 12:55 PM Igor Sapego <isap...@apache.org> wrote: >> >> > Pavel, >> > >> > I've checked the IEP and I like it. The only thing that seems a bit >> > confusing to me >> > is that there are 4 different variants for clients but there are cons >> and >> > pros for >> > different variants. Maybe at least few sentences should be written here >> to >> > give developers who are not familiar with DataStreamer some ideas and >> make >> > it easier for them to choose. >> > >> > Best Regards, >> > Igor >> > >> > >> > On Thu, Mar 4, 2021 at 3:14 PM Pavel Tupitsyn <ptupit...@apache.org> >> > wrote: >> > >> > > Igniters, >> > > >> > > Please review the IEP [1] and let me know what you think. >> > > >> > > [1] >> > > >> > > >> > >> https://cwiki.apache.org/confluence/display/IGNITE/IEP-68%3A+Thin+Client+Data+Streamer >> > > >> > >> > > > -- > Sincerely yours, Ivan Daschinskiy > -- Sincerely yours, Ivan Daschinskiy
