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 C3310200BE3 for ; Thu, 22 Dec 2016 20:55:28 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id C1C1E160B26; Thu, 22 Dec 2016 19:55:28 +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 0C006160B1B for ; Thu, 22 Dec 2016 20:55:27 +0100 (CET) Received: (qmail 89600 invoked by uid 500); 22 Dec 2016 19:55:27 -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 89589 invoked by uid 99); 22 Dec 2016 19:55:27 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 22 Dec 2016 19:55:27 +0000 Received: from [192.168.75.153] (c-73-222-138-29.hsd1.ca.comcast.net [73.222.138.29]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id E3E361A028B for ; Thu, 22 Dec 2016 19:55:26 +0000 (UTC) From: Denis Magda Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Mime-Version: 1.0 (Mac OS X Mail 10.2 \(3259\)) Subject: Re: Node attribute filter Date: Thu, 22 Dec 2016 11:55:26 -0800 References: To: dev@ignite.apache.org In-Reply-To: Message-Id: <3595FF1E-B107-4DC0-8276-E542B0C9221A@apache.org> X-Mailer: Apple Mail (2.3259) archived-at: Thu, 22 Dec 2016 19:55:28 -0000 Val, thanks. Reviewed. Looks good to merge. =E2=80=94 Denis > On Dec 20, 2016, at 3:27 PM, Valentin Kulichenko = wrote: >=20 > Guys, >=20 > I created a simple implementation of IgnitePredicate = based on > user attributes. This is the most commonly used approach for node = filters, > but currently users are forced to create their own implementations = each > time. Would be useful to have it out of the box. >=20 > Can someone take a look at the implementation and tell me if there is > anything wrong/missed? >=20 > Ticket: https://issues.apache.org/jira/browse/IGNITE-4439 > PR: https://github.com/apache/ignite/pull/1369 >=20 > -Val