Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id BC021200BEF for ; Wed, 21 Dec 2016 02:33:36 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id BA9AE160B33; Wed, 21 Dec 2016 01:33:36 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 1207C160B29 for ; Wed, 21 Dec 2016 02:33:35 +0100 (CET) Received: (qmail 39136 invoked by uid 500); 21 Dec 2016 01:33:35 -0000 Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ignite.apache.org Delivered-To: mailing list dev@ignite.apache.org Received: (qmail 39122 invoked by uid 99); 21 Dec 2016 01:33:35 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Dec 2016 01:33:35 +0000 Received: from mail-qt0-f176.google.com (mail-qt0-f176.google.com [209.85.216.176]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id E46901A0046 for ; Wed, 21 Dec 2016 01:33:34 +0000 (UTC) Received: by mail-qt0-f176.google.com with SMTP id n6so193805500qtd.1 for ; Tue, 20 Dec 2016 17:33:34 -0800 (PST) X-Gm-Message-State: AIkVDXKGE0Z3eDGu3LZGe0ctTH0RJihJkzeCtLAIJf13ILB6s3ml62HU7TprtB8db94VdkWN/gq8DGdalAUPn47H X-Received: by 10.200.37.221 with SMTP id f29mr2548954qtf.123.1482284013968; Tue, 20 Dec 2016 17:33:33 -0800 (PST) MIME-Version: 1.0 Received: by 10.237.62.119 with HTTP; Tue, 20 Dec 2016 17:32:53 -0800 (PST) In-Reply-To: References: From: Dmitriy Setrakyan Date: Tue, 20 Dec 2016 17:32:53 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: Make async API great again To: dev@ignite.apache.org Content-Type: multipart/alternative; boundary=001a113f41a81c1ce70544212424 archived-at: Wed, 21 Dec 2016 01:33:36 -0000 --001a113f41a81c1ce70544212424 Content-Type: text/plain; charset=UTF-8 Would be nice if someone would prototype a new cache API and post the generated javadoc here. I think we all will benefit from reviewing it. On Tue, Dec 20, 2016 at 12:17 PM, Vladimir Ozerov wrote: > Async API rework is mechanical addition of ~100 methods through copy-paste. > Should not take more than a day to implement and more than another day to > rework tests. > > On Tue, Dec 20, 2016 at 10:00 PM, Dmitriy Setrakyan > > wrote: > > > How difficult is this change? Does not look like it can be done > overnight. > > > > On Tue, Dec 20, 2016 at 10:46 AM, Vladimir Ozerov > > wrote: > > > > > We already discussed this several months ago in other thread. > > > > > > "Async" methods is the most simple and straight API possible. .NET > world > > > goes this way all over their frameworks and nobody died. Hazelcast also > > > goes this way. Java goes this way (see CompletableFuture). This is > common > > > and well-known practice. The most impacted part of our API will be > cache, > > > +33 new methods. Though, I do not see how it can affect learning curve. > > > > > > Agree that we should deprecate AsyncSupport gradually and remove it no > > > earlier than in Apache Ignite 3.0. > > > > > > On Tue, Dec 20, 2016 at 9:31 PM, Dmitriy Setrakyan < > > dsetrakyan@apache.org> > > > wrote: > > > > > > > On Tue, Dec 20, 2016 at 10:28 AM, Sergi Vladykin < > > > sergi.vladykin@gmail.com > > > > > > > > > wrote: > > > > > > > > > +1 For removing withAsync. It is a broken design. > > > > > > > > > > > > > Sergi, do you also want to add all the async methods to the main API > or > > > do > > > > you have some other design in mind? > > > > > > > > > > --001a113f41a81c1ce70544212424--