Return-Path: X-Original-To: apmail-apex-dev-archive@minotaur.apache.org Delivered-To: apmail-apex-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 833B818B04 for ; Tue, 27 Oct 2015 16:40:29 +0000 (UTC) Received: (qmail 42713 invoked by uid 500); 27 Oct 2015 16:40:29 -0000 Delivered-To: apmail-apex-dev-archive@apex.apache.org Received: (qmail 42659 invoked by uid 500); 27 Oct 2015 16:40:29 -0000 Mailing-List: contact dev-help@apex.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@apex.incubator.apache.org Delivered-To: mailing list dev@apex.incubator.apache.org Received: (qmail 42648 invoked by uid 99); 27 Oct 2015 16:40:29 -0000 Received: from Unknown (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 Oct 2015 16:40:29 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id C3DF71809D1 for ; Tue, 27 Oct 2015 16:40:28 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.971 X-Spam-Level: X-Spam-Status: No, score=0.971 tagged_above=-999 required=6.31 tests=[KAM_LAZY_DOMAIN_SECURITY=1, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, T_RP_MATCHES_RCVD=-0.01, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-eu-west.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id aPckrXnGCUfB for ; Tue, 27 Oct 2015 16:40:19 +0000 (UTC) Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx1-eu-west.apache.org (ASF Mail Server at mx1-eu-west.apache.org) with SMTP id 1606520924 for ; Tue, 27 Oct 2015 16:40:18 +0000 (UTC) Received: (qmail 41851 invoked by uid 99); 27 Oct 2015 16:40:18 -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, 27 Oct 2015 16:40:18 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 33C54DFE13; Tue, 27 Oct 2015 16:40:18 +0000 (UTC) From: tweise To: dev@apex.incubator.apache.org Reply-To: dev@apex.incubator.apache.org References: In-Reply-To: Subject: [GitHub] incubator-apex-core pull request: - APEX-225 #resolve Adds isConne... Content-Type: text/plain Message-Id: <20151027164018.33C54DFE13@git1-us-west.apache.org> Date: Tue, 27 Oct 2015 16:40:18 +0000 (UTC) Github user tweise commented on the pull request: https://github.com/apache/incubator-apex-core/pull/139#issuecomment-151562918 The InputPort interface is for the engine. The engine already knows whether the port is connected, hence it calls setConnected. There is no need to have isConnected in this interface. DefaultInputPort can expose the result of setConnected to the operator side, I don't see a reason why not. --- 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. ---