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 5A0DB200B9B for ; Wed, 28 Sep 2016 07:11:58 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 58888160AE5; Wed, 28 Sep 2016 05:11:58 +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 9F05D160AD2 for ; Wed, 28 Sep 2016 07:11:57 +0200 (CEST) Received: (qmail 81895 invoked by uid 500); 28 Sep 2016 05:11:56 -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 81884 invoked by uid 99); 28 Sep 2016 05:11:56 -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 05:11:56 +0000 Received: from mail-qk0-f174.google.com (mail-qk0-f174.google.com [209.85.220.174]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id 765311A018B for ; Wed, 28 Sep 2016 05:11:56 +0000 (UTC) Received: by mail-qk0-f174.google.com with SMTP id g67so37446030qkd.0 for ; Tue, 27 Sep 2016 22:11:56 -0700 (PDT) X-Gm-Message-State: AA6/9Rnu9EiVzkRRnrtL7sgkYyktuPHX3FjMtOra+8pAZkIqhdeWwMPM+RG1Zxw2rYKwrNb2Ur2iMOaxv42mug== X-Received: by 10.55.26.87 with SMTP id a84mr11056909qka.187.1475039515644; Tue, 27 Sep 2016 22:11:55 -0700 (PDT) MIME-Version: 1.0 Received: by 10.200.37.9 with HTTP; Tue, 27 Sep 2016 22:11:55 -0700 (PDT) In-Reply-To: References: From: Preston Carman Date: Tue, 27 Sep 2016 22:11:55 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: Physical Operator Tag Change (SPLIT -> REPLICATE) To: dev@asterixdb.apache.org Content-Type: text/plain; charset=UTF-8 archived-at: Wed, 28 Sep 2016 05:11:58 -0000 I always thought it was confusing that the ReplicatePOperator used the SPLIT tag. I think making the name consistent would be nice. 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 >>