Re: [DISCUSS] IEP-71 Public API for secondary index search

2021-08-30 Thread Maksim Timonin
Hi! It's a PR re-review request. I've reworked PR [1]. The major changes are: 1. indexName is a required param for building IndexQuery. 2. the order of criteria fields doesn't make sense - now it checks only that criteria is a prefix for index fields. 3. IndexQuery can accept a value type (only v

Re: Re[2]: Google Guava in Ignite 3

2021-08-30 Thread Andrey Gura
Follow up On Mon, Aug 23, 2021 at 1:22 PM Andrey Gura wrote: > > Igniters, > > What I actually didn't understand from this thread: Is Guava allowed > in production code of Ignite 3 modules (not dependencies like > Calcite)? > > While we agree with using shading I don't see any arguments about > u