Pu/lookupfix

[izulin]

Context

Fixes #787

How has this been tested?

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature or improvement (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Additional language file or change to the existing one (translations)

Related issue(s):

Checklist:

• My code follows the code style of this project,
• My change requires a change to the documentation,
• I described the modification in the CHANGELOG.md file.

[codecov]

Codecov Report

Merging #791 (06ff3e5) into develop (b303691) will decrease coverage by 0.01%.
The diff coverage is 58.82%.

@@             Coverage Diff             @@
##           develop     #791      +/-   ##
===========================================
- Coverage    92.35%   92.34%   -0.02%     
===========================================
  Files          166      166              
  Lines        40339    40347       +8     
  Branches      3722     3719       -3     
===========================================
  Hits         37257    37257              
- Misses        3076     3084       +8     
  Partials         6        6              

Impacted Files	Coverage Δ
src/Config.ts	98.63% <44.44%> (-0.27%)	⬇️
src/interpreter/binarySearch.ts	68.75% <66.66%> (-5.98%)	⬇️
src/Lookup/ColumnBinarySearch.ts	93.93% <100.00%> (+1.51%)	⬆️
src/Lookup/RowSearchStrategy.ts	85.00% <100.00%> (ø)

[wojciechczerniak]

👍

[wojciechczerniak]

Can you add deprecated tag to the binarySearchThreshold option description?
ie * @deprecated Every search of sorted data always uses binary search

[izulin]

in the Config?

[wojciechczerniak]

yes, it will be displayed in the API reference