Igniters,
In *IgniteAtomicLong* I see the following statement: "All previously described methods have asynchronous analogs." But there are no async operations in it's implemenation. Is this a bug in documentation of we forgot to implement them? Vladimir. |
There is the same comment for the following interfaces:
- IgniteAtomicReference - IgniteAtomicSequence - IgniteAtomicStamped On Thu, Sep 17, 2015 at 3:00 PM, Vladimir Ozerov <[hidden email]> wrote: > Igniters, > > In *IgniteAtomicLong* I see the following statement: "All previously > described methods have asynchronous analogs." But there are no async > operations in it's implemenation. > > Is this a bug in documentation of we forgot to implement them? > > Vladimir. > -- Andrey Gura GridGain Systems, Inc. www.gridgain.com |
In this case, should we just simply update the documentation to reflect the
reality? On Thu, Sep 17, 2015 at 3:30 PM, Andrey Gura <[hidden email]> wrote: > There is the same comment for the following interfaces: > > - IgniteAtomicReference > - IgniteAtomicSequence > - IgniteAtomicStamped > > On Thu, Sep 17, 2015 at 3:00 PM, Vladimir Ozerov <[hidden email]> > wrote: > > > Igniters, > > > > In *IgniteAtomicLong* I see the following statement: "All previously > > described methods have asynchronous analogs." But there are no async > > operations in it's implemenation. > > > > Is this a bug in documentation of we forgot to implement them? > > > > Vladimir. > > > > > > -- > Andrey Gura > GridGain Systems, Inc. > www.gridgain.com > |
Absolutely agree ;)
Andrey, can you please do? --Yakov 2015-09-17 17:17 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > In this case, should we just simply update the documentation to reflect the > reality? > > On Thu, Sep 17, 2015 at 3:30 PM, Andrey Gura <[hidden email]> wrote: > > > There is the same comment for the following interfaces: > > > > - IgniteAtomicReference > > - IgniteAtomicSequence > > - IgniteAtomicStamped > > > > On Thu, Sep 17, 2015 at 3:00 PM, Vladimir Ozerov <[hidden email]> > > wrote: > > > > > Igniters, > > > > > > In *IgniteAtomicLong* I see the following statement: "All previously > > > described methods have asynchronous analogs." But there are no async > > > operations in it's implemenation. > > > > > > Is this a bug in documentation of we forgot to implement them? > > > > > > Vladimir. > > > > > > > > > > > -- > > Andrey Gura > > GridGain Systems, Inc. > > www.gridgain.com > > > |
Done
https://github.com/apache/ignite/pull/100 On Fri, Sep 18, 2015 at 12:36 PM, Yakov Zhdanov <[hidden email]> wrote: > Absolutely agree ;) > > Andrey, can you please do? > > --Yakov > > 2015-09-17 17:17 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > In this case, should we just simply update the documentation to reflect > the > > reality? > > > > On Thu, Sep 17, 2015 at 3:30 PM, Andrey Gura <[hidden email]> wrote: > > > > > There is the same comment for the following interfaces: > > > > > > - IgniteAtomicReference > > > - IgniteAtomicSequence > > > - IgniteAtomicStamped > > > > > > On Thu, Sep 17, 2015 at 3:00 PM, Vladimir Ozerov <[hidden email] > > > > > wrote: > > > > > > > Igniters, > > > > > > > > In *IgniteAtomicLong* I see the following statement: "All previously > > > > described methods have asynchronous analogs." But there are no async > > > > operations in it's implemenation. > > > > > > > > Is this a bug in documentation of we forgot to implement them? > > > > > > > > Vladimir. > > > > > > > > > > > > > > > > -- > > > Andrey Gura > > > GridGain Systems, Inc. > > > www.gridgain.com > > > > > > -- Andrey Gura GridGain Systems, Inc. www.gridgain.com |
I reviewed the changes and I agree. Anton V. can you please merge this PR?
--Yakov 2015-09-18 15:26 GMT+03:00 Andrey Gura <[hidden email]>: > Done > > https://github.com/apache/ignite/pull/100 > > On Fri, Sep 18, 2015 at 12:36 PM, Yakov Zhdanov <[hidden email]> > wrote: > > > Absolutely agree ;) > > > > Andrey, can you please do? > > > > --Yakov > > > > 2015-09-17 17:17 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > > > In this case, should we just simply update the documentation to reflect > > the > > > reality? > > > > > > On Thu, Sep 17, 2015 at 3:30 PM, Andrey Gura <[hidden email]> > wrote: > > > > > > > There is the same comment for the following interfaces: > > > > > > > > - IgniteAtomicReference > > > > - IgniteAtomicSequence > > > > - IgniteAtomicStamped > > > > > > > > On Thu, Sep 17, 2015 at 3:00 PM, Vladimir Ozerov < > [hidden email] > > > > > > > wrote: > > > > > > > > > Igniters, > > > > > > > > > > In *IgniteAtomicLong* I see the following statement: "All > previously > > > > > described methods have asynchronous analogs." But there are no > async > > > > > operations in it's implemenation. > > > > > > > > > > Is this a bug in documentation of we forgot to implement them? > > > > > > > > > > Vladimir. > > > > > > > > > > > > > > > > > > > > > -- > > > > Andrey Gura > > > > GridGain Systems, Inc. > > > > www.gridgain.com > > > > > > > > > > > > > -- > Andrey Gura > GridGain Systems, Inc. > www.gridgain.com > |
Done
On Fri, Sep 18, 2015 at 11:45 PM, Yakov Zhdanov <[hidden email]> wrote: > I reviewed the changes and I agree. Anton V. can you please merge this PR? > > --Yakov > > 2015-09-18 15:26 GMT+03:00 Andrey Gura <[hidden email]>: > > > Done > > > > https://github.com/apache/ignite/pull/100 > > > > On Fri, Sep 18, 2015 at 12:36 PM, Yakov Zhdanov <[hidden email]> > > wrote: > > > > > Absolutely agree ;) > > > > > > Andrey, can you please do? > > > > > > --Yakov > > > > > > 2015-09-17 17:17 GMT+03:00 Dmitriy Setrakyan <[hidden email]>: > > > > > > > In this case, should we just simply update the documentation to > reflect > > > the > > > > reality? > > > > > > > > On Thu, Sep 17, 2015 at 3:30 PM, Andrey Gura <[hidden email]> > > wrote: > > > > > > > > > There is the same comment for the following interfaces: > > > > > > > > > > - IgniteAtomicReference > > > > > - IgniteAtomicSequence > > > > > - IgniteAtomicStamped > > > > > > > > > > On Thu, Sep 17, 2015 at 3:00 PM, Vladimir Ozerov < > > [hidden email] > > > > > > > > > wrote: > > > > > > > > > > > Igniters, > > > > > > > > > > > > In *IgniteAtomicLong* I see the following statement: "All > > previously > > > > > > described methods have asynchronous analogs." But there are no > > async > > > > > > operations in it's implemenation. > > > > > > > > > > > > Is this a bug in documentation of we forgot to implement them? > > > > > > > > > > > > Vladimir. > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > Andrey Gura > > > > > GridGain Systems, Inc. > > > > > www.gridgain.com > > > > > > > > > > > > > > > > > > > > -- > > Andrey Gura > > GridGain Systems, Inc. > > www.gridgain.com > > > |
Free forum by Nabble | Edit this page |