Got it, thanks! Just submitted a patch to fix it. Also, the same problem exists when we upsert into a secondary index. Only missing values are filtered out, but not null values.
Best regards, Chen Luo On Sun, May 27, 2018 at 1:27 PM, Mike Carey <[email protected]> wrote: > Indeed. > > On Sun, May 27, 2018, 11:40 AM abdullah alamoudi <[email protected]> > wrote: > > > I think that this would be a bug if that was the case. I am pretty sure > at > > some point, we had a filter for nulls. > > > > Cheers, > > Abdullah. > > > > > On May 26, 2018, at 5:25 AM, Chen Luo <[email protected]> wrote: > > > > > > Hi devs, > > > > > > I've a question about null values in secondary index. I've assumed that > > > null values are simply excluded from the secondary index. For example, > > if I > > > build a secondary index on Name, then a record with Name being null > would > > > not be indexed. I also saw in IntroduceSecondaryIndexInsertDeleteRule > we > > > created some filtering expression to filter out null secondary values. > > > > > > However, when we bulkload a dataset, this filtering expression is never > > > executed when loading a secondary index, which implies all null values > > are > > > added to the secondary index. Is it safe to assume this is a bug? > > > > > > Best regards, > > > Chen Luo > > > > >
