Re: [DISCUSS] Sarg (search argument) to generalize and replace IN in RexCall

[Julian Hyde]

> On Sep 29, 2020, at 11:40 AM, Vladimir Sitnikov <sitnikov.vladi...@gmail.com> 
> wrote:
> 
> Julian>Suppose - worst case scenario - that Guava 30 removes RangeSet. The
> solution would be for us to copy RangeSet and enough dependent classes into
> Calcite to serve our needs
> 
> Then all the clients would have to adjust packages because we can't copy
> Guava code and keep Guava's package names.

That is a valid concern. RangeSet used in a public field and a public method in 
class Sarg.

The vast majority of clients who use Sarg (or expressions that contain Sarg) 
will not reference RangeSet directly and therefore would not be impacted. So I 
think it’s an acceptable risk.

Julian