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 C8F2A200AC8 for ; Tue, 7 Jun 2016 19:14:37 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id C7942160A36; Tue, 7 Jun 2016 17:14:37 +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 1A080160968 for ; Tue, 7 Jun 2016 19:14:36 +0200 (CEST) Received: (qmail 70789 invoked by uid 500); 7 Jun 2016 17:14:36 -0000 Mailing-List: contact dev-help@quickstep.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@quickstep.incubator.apache.org Delivered-To: mailing list dev@quickstep.incubator.apache.org Received: (qmail 70778 invoked by uid 99); 7 Jun 2016 17:14:36 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd4-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 07 Jun 2016 17:14:36 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd4-us-west.apache.org (ASF Mail Server at spamd4-us-west.apache.org) with ESMTP id AC699C0773 for ; Tue, 7 Jun 2016 17:14:35 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd4-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -5.446 X-Spam-Level: X-Spam-Status: No, score=-5.446 tagged_above=-999 required=6.31 tests=[KAM_LAZY_DOMAIN_SECURITY=1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.426] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd4-us-west.apache.org [10.40.0.11]) (amavisd-new, port 10024) with ESMTP id HC8Z3jeIm9Vy for ; Tue, 7 Jun 2016 17:14:33 +0000 (UTC) Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with SMTP id 3F6B85FAD5 for ; Tue, 7 Jun 2016 17:14:33 +0000 (UTC) Received: (qmail 70768 invoked by uid 99); 7 Jun 2016 17:14:32 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 07 Jun 2016 17:14:32 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 6A5F7DFC7E; Tue, 7 Jun 2016 17:14:32 +0000 (UTC) From: pateljm To: dev@quickstep.incubator.apache.org Reply-To: dev@quickstep.incubator.apache.org References: In-Reply-To: Subject: [GitHub] incubator-quickstep issue #8: Added query id in all the operators and the wo... Content-Type: text/plain Message-Id: <20160607171432.6A5F7DFC7E@git1-us-west.apache.org> Date: Tue, 7 Jun 2016 17:14:32 +0000 (UTC) archived-at: Tue, 07 Jun 2016 17:14:38 -0000 Github user pateljm commented on the issue: https://github.com/apache/incubator-quickstep/pull/8 @hbdeshmukh Ok -- I see your point that the style guide contradicts putting the query_id first. How about we close this PR, and have a separate discussion on changes we need to make to the style guide. @zuyu Any comments on small change to the style guide to allow these changes? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastructure@apache.org or file a JIRA ticket with INFRA. ---