Hi Igor, Could you please help with answering to this question?
Sincerely, Dmitriy Pavlov вт, 15 мая 2018 г. в 5:18, Sunny Chan, CLSA <[email protected]>: > Hello, > > > > In Ignite-5779 patch, CassandraSessionImpl.java line 289 a null check for row > has been removed (before the change: > https://github.com/apache/ignite/blob/924b1faa64026107bf933ba441e743cf52cb94d1/modules/cassandra/store/src/main/java/org/apache/ignite/cache/store/cassandra/session/CassandraSessionImpl.java#L289 > > ) > > > > I cannot tell immediately why the null check is no longer needed, > especially the row assignment in line 289 can definitely return a null. Is > it a mistake or we are expecting BatchExecutionAssistant is able to deal > with null? > > > > Thanks. > > > > *Sunny Chan* > > *Senior Lead Engineer, Executive Services* > > D +852 2600 8907 <+852%202600%208907> | M +852 6386 1835 > <+852%206386%201835> | T +852 2600 8888 <+852%202600%208888> > > 5/F, One Island East, 18 Westlands Road > <https://maps.google.com/?q=18+Westlands+Road&entry=gmail&source=g>, > Island East, Hong Kong > > > > [image: :1. Social Media Icons:CLSA_Social Media Icons_linkedin.png] > <https://hk.linkedin.com/company/clsa>[image: :1. Social Media > Icons:CLSA_Social Media Icons_twitter.png] > <https://twitter.com/clsainsights?lang=en>[image: :1. Social Media > Icons:CLSA_Social Media Icons_youtube.png] > <https://www.youtube.com/channel/UC0qWp_lLnOcRYmBlCNQgZKA>[image: :1. > Social Media Icons:CLSA_Social Media Icons_facebook.png] > <https://www.facebook.com/clsacommunity/> > > > > *clsa.com* <https://www.clsa.com/> > > *Insights. Liquidity. Capital. * > > > > [image: CLSA_RGB] <https://www.clsa.com/member> > > > > *A CITIC Securities Company* > > > > The content of this communication is intended for the recipient and is > subject to CLSA Legal and Regulatory Notices. > These can be viewed at https://www.clsa.com/disclaimer.html or sent to > you upon request. > Please consider before printing. CLSA is ISO14001 certified and committed > to reducing its impact on the environment. >
