OK, let's do it this way.
2017-01-13 13:27 GMT+03:00 Sergi Vladykin <[email protected]>: > Internally we still use int[] when we send partitions (see > GridH2QueryRequest.parts). It looks like we only do more work with > PartitionSet. > > I like the idea of bitset for partitions, but > > 1. We have to change internals first to use it, otherwise the optimization > makes no sense. > 2. We will need to have a method SqlQuery.setPartitions(int... parts) for > usability reasons anyways. > > Thus I suggest for now to go the straightforward way with int[] and create > a separate ticket describing the optimization with bitset. > > Sergi > > 2017-01-13 13:06 GMT+03:00 Alexei Scherbakov <[email protected] > >: > > > PartitionSet hides internal implementation of int array. > > > > This allows as to efficiently represent contiguous range of partitions > and > > defines clear API for ordered iteration over partitions and containment > > check. > > > > Even better to go with compressed bitmap, as I mentioned in ticket > comment. > > This will allow us to minimize heap footprint for this object. > > > > Moreover, it will be useful to create reusable compressed bitmap > > implementation in Ignite and use it in other cases, on example, for > > replacing H2's IntArray and Set<Integer>. > > > > Should I create a ticket for this ? > > > > . > > > > 2017-01-13 1:01 GMT+03:00 Dmitriy Setrakyan <[email protected]>: > > > > > On Thu, Jan 12, 2017 at 6:12 AM, Sergi Vladykin < > > [email protected]> > > > wrote: > > > > > > > I looked at the code. The PartitionSet concept looks overengineered > to > > > me, > > > > why wouldn't we just go with int[]? > > > > > > > > > > Agree. > > > > > > > > > > > > > > Sergi > > > > > > > > 2017-01-12 15:18 GMT+03:00 Alexei Scherbakov < > > > [email protected] > > > > >: > > > > > > > > > Done. > > > > > > > > > > 2017-01-11 20:39 GMT+03:00 Dmitriy Setrakyan < > [email protected] > > >: > > > > > > > > > > > Alexey, > > > > > > > > > > > > I am not sure I am seeing the API changes documented in the > ticket. > > > Can > > > > > you > > > > > > please either document them or add GIT links for the new classes? > > > > > > > > > > > > D. > > > > > > > > > > > > On Wed, Jan 11, 2017 at 9:29 AM, Alexei Scherbakov < > > > > > > [email protected]> wrote: > > > > > > > > > > > > > Guys, > > > > > > > > > > > > > > I've just submitted a PR for > > > > > > > https://issues.apache.org/jira/browse/IGNITE-4523. > > > > > > > > > > > > > > Please review API changes while waiting for TC results. > > > > > > > > > > > > > > -- > > > > > > > > > > > > > > Best regards, > > > > > > > Alexei Scherbakov > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > Best regards, > > > > > Alexei Scherbakov > > > > > > > > > > > > > > > > > > > > -- > > > > Best regards, > > Alexei Scherbakov > > > -- Best regards, Alexei Scherbakov
