Replace dictionary proxies with nested dictionaries 14/N

intel / hdk

A low-level execution library for analytic data processing.

Apache License 2.0

31 stars 14 forks source link

Replace dictionary proxies with nested dictionaries 14/N #699

Closed ienkovich closed 1 year ago

ienkovich commented 1 year ago

Support nested dictionaries in StringDictionary::getCompare.

This patch also fixes several bugs in this method. The logic around the comparison of cache_index->index with zero is completely broken because the zero index has no special meaning at all. Added tests cover both nested dictionaries and these fixed bugs.