Sort values before the search

[rui-mo]

Description

I notice unexpected result for testing range when values are not sorted.

velox/velox/type/Filter.cpp

Lines 1089 to 1092 in 386fed7

	auto lo = std::lower_bound(
	nonNegated_->values().begin(), nonNegated_->values().end(), min);
	auto hi = std::lower_bound(
	nonNegated_->values().begin(), nonNegated_->values().end(), max);

It appears that values are not meant to be pre-sorted as the parameter of 'NegatedBigintValuesUsingHashTable', based on the implementation of 'ExprToSubfieldFilterParser::makeInFilter'. I believe that sorting values before searching with 'lower_bound' will resolve this problem. If this is the case, I'd be happy to open a fix for it.

[rui-mo]

@Yuhta Would you help share your thoughts? Thanks.