Return-Path: X-Original-To: apmail-jackrabbit-oak-dev-archive@minotaur.apache.org Delivered-To: apmail-jackrabbit-oak-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 2C9A0E10D for ; Fri, 22 Feb 2013 12:21:09 +0000 (UTC) Received: (qmail 11186 invoked by uid 500); 22 Feb 2013 12:21:08 -0000 Delivered-To: apmail-jackrabbit-oak-dev-archive@jackrabbit.apache.org Received: (qmail 10957 invoked by uid 500); 22 Feb 2013 12:21:04 -0000 Mailing-List: contact oak-dev-help@jackrabbit.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: oak-dev@jackrabbit.apache.org Delivered-To: mailing list oak-dev@jackrabbit.apache.org Received: (qmail 10885 invoked by uid 99); 22 Feb 2013 12:21:01 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 22 Feb 2013 12:21:01 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of jukka.zitting@gmail.com designates 209.85.214.171 as permitted sender) Received: from [209.85.214.171] (HELO mail-ob0-f171.google.com) (209.85.214.171) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 22 Feb 2013 12:20:55 +0000 Received: by mail-ob0-f171.google.com with SMTP id x4so518771obh.2 for ; Fri, 22 Feb 2013 04:20:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:mime-version:in-reply-to:references:from:date:message-id :subject:to:content-type; bh=qbYOrMEpsO900TJK+ixZOWbkpw8FCNNn+wpS8o5RGQ0=; b=DOKfHSYjducoFX+yDE0ajDsY8xnoVIXHebw9lMFt+aImunQryh8NSNY2zU11nAPlVO uXWt3Yk4gUhV6K3zEoaRU6telNBcRetI3MqZfFcJqEvnjWmVWpDtJ9eJTpwUko+piUOO Ppyz3lEv9myQwUrNY43yxK6CH+QU5DBBRx6KxgsOQ0pNvuDoqSyEKdTKey/ce7URTl3q bi52fOrdpGkGeBErO33GAuWqQkKHSebm0iyZRynGVW7mTgvlQX9U+yCsuOokgd986gYg du9Pesqm+OzS6sIOujrN7L8yDtwrni72RfQ39Ht5TuBO5Ki2r9z4wg0nWb1QHmfVt3H1 ejhA== X-Received: by 10.60.172.18 with SMTP id ay18mr668400oec.126.1361535634583; Fri, 22 Feb 2013 04:20:34 -0800 (PST) MIME-Version: 1.0 Received: by 10.182.47.35 with HTTP; Fri, 22 Feb 2013 04:20:14 -0800 (PST) In-Reply-To: References: <4C2FF119-6C19-4E46-BF8B-4164A6C13132@adobe.com> From: Jukka Zitting Date: Fri, 22 Feb 2013 14:20:14 +0200 Message-ID: Subject: Re: One hook to rule them all To: Oak devs Content-Type: text/plain; charset=ISO-8859-1 X-Virus-Checked: Checked by ClamAV on apache.org Hi, On Fri, Feb 22, 2013 at 11:28 AM, Alex Parvulescu wrote: > I almost had this going until I ran into some CommitHooks that deviate from > this standard behavior (like the ValidatingHook for ex which needs a root > validator) Actually you could think of the ValidatingHook as a variant of your approach, as it also maintains a set of diff objects (Validator is much like NodeStateDiff). The main difference to the index hook mechanism is that validators can't modify the content tree. I'm hoping to come up with an approach that could generalize and merge these features into one hook. BR, Jukka Zitting