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 5A51E200B9B for ; Wed, 28 Sep 2016 05:44:22 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 58C30160AE5; Wed, 28 Sep 2016 03:44:22 +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 C5DDD160AD2 for ; Wed, 28 Sep 2016 05:44:21 +0200 (CEST) Received: (qmail 31994 invoked by uid 500); 28 Sep 2016 03:44:20 -0000 Mailing-List: contact dev-help@asterixdb.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@asterixdb.apache.org Delivered-To: mailing list dev@asterixdb.apache.org Received: (qmail 31973 invoked by uid 99); 28 Sep 2016 03:44:20 -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, 28 Sep 2016 03:44:20 +0000 Received: from [192.168.1.82] (172-10-235-13.lightspeed.sntcca.sbcglobal.net [172.10.235.13]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id 61A5B1A018B for ; Wed, 28 Sep 2016 03:44:20 +0000 (UTC) From: "Till Westmann" To: dev@asterixdb.apache.org Subject: Re: Physical Operator Tag Change (SPLIT -> REPLICATE) Date: Tue, 27 Sep 2016 20:44:19 -0700 Message-ID: In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; markup=markdown X-Mailer: MailMate (1.9.5r5263) archived-at: Wed, 28 Sep 2016 03:44:22 -0000 Looks good, ready to merge. Cheers, Till On 27 Sep 2016, at 17:11, Taewoo Kim wrote: > @Yingyi: Thanks for the suggestion. Actually, I have crated a separate > patch for this change only: https://asterix-gerrit.ics.uci.edu/#/c/1219/. > Once this is merged, the review for the second step would be much easier. > > Best, > Taewoo > > On Tue, Sep 27, 2016 at 5:08 PM, Yingyi Bu wrote: > >> Taewoo, >> >> I agree with you. >> However, can you have two separate changes: >> 1. Your current change https://asterix-gerrit.ics.uci.edu/#/c/1196/. Call >> your "Split" "PartitionSplit" for now. >> 2. Do what you proposed in the next change which only does the renaming. >> >> That makes reviews less overwhelming and easier. >> >> Best, >> Yingyi >> >> >> On Tue, Sep 27, 2016 at 5:01 PM, Taewoo Kim wrote: >> >>> Hello All, >>> >>> Regarding the index-only plan, I am going to introduce an operator named >>> SplitOperator that propagates each tuple to only one output branch. >>> Currently, ReplicateOperator propagates each tuple to all output >> branches. >>> And ReplicatePOperator uses the physical operator tag as SPLIT. I suggest >>> we change it to REPLICATE. Thus, SPLIT tag can be used for real Split >>> operator. Any thoughts? >>> >>> Best, >>> Taewoo >>> >>