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 AFED8CFC8 for ; Fri, 27 Apr 2012 10:40:15 +0000 (UTC) Received: (qmail 20535 invoked by uid 500); 27 Apr 2012 10:40:15 -0000 Delivered-To: apmail-jackrabbit-oak-dev-archive@jackrabbit.apache.org Received: (qmail 20511 invoked by uid 500); 27 Apr 2012 10:40:15 -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 20503 invoked by uid 99); 27 Apr 2012 10:40:15 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 27 Apr 2012 10:40:15 +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 74.125.82.170 as permitted sender) Received: from [74.125.82.170] (HELO mail-we0-f170.google.com) (74.125.82.170) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 27 Apr 2012 10:40:09 +0000 Received: by werh12 with SMTP id h12so785735wer.1 for ; Fri, 27 Apr 2012 03:39:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type:content-transfer-encoding; bh=x74R9lQ/N1cYaVOrpJwWGEGNSm+/L2mk6+2jI48xQic=; b=ggKThJIK7AhapqdC149jvmSTTzFafDY8kLa5xWn4gUSAvX89ylP5lNzFHEsuu0JvjL /mpzp6ar+0O622r/sMOjsvUjzOVnaT6uHSK7JXDeaJ91ETr4BAIwFpN6By7CfVSmBcf4 GDOgKjR0kw91XXnlY9tknnzNSDNF5Yhz2eDgDfD5T5Y+A74TmJE1c6V3C1A/G899/4+Z yKsMVTOS4OQUZyMryBQ+xiciGp91k0LcpVRfVZlJxvdrWcE06Es3TseaoWhmEjZ9xCq0 eKZDXWEm7CWPos4UPp3L5X2nAU2yuvOrgbifGacutQUCRbx9u4WSfqOG2H6L14CaUqPU c3JA== Received: by 10.216.135.141 with SMTP id u13mr93873wei.79.1335523187925; Fri, 27 Apr 2012 03:39:47 -0700 (PDT) MIME-Version: 1.0 Received: by 10.180.146.133 with HTTP; Fri, 27 Apr 2012 03:39:27 -0700 (PDT) In-Reply-To: <4F9A7006.2050802@adobe.com> References: <4F9A7006.2050802@adobe.com> From: Jukka Zitting Date: Fri, 27 Apr 2012 12:39:27 +0200 Message-ID: Subject: Re: Consolidate oak Utilities To: oak-dev@jackrabbit.apache.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-Virus-Checked: Checked by ClamAV on apache.org Hi, On Fri, Apr 27, 2012 at 12:08 PM, Angela Schreiber wrot= e: > a) create oak-commons module and move the various utilities > =A0 used in multiple projects there... including merging > =A0 redundant/duplicate/very-similar code. +1 for cases where there's obvious duplication of code across components. However, to me many such cases seem like warning signs of potentially broken interface design, so it would be good to review such cases (possibly after they've been moved to a shared commons component). For example (and I know this has been discussed at length already), much of the path and json/p handling code shared between -core and -mk is just there to first generate jsop strings and then parse them again. It's obviously good to avoid duplicating such code, but even better if we could avoid it in the first place. > b) move the utilities to jcr-commons. since we use jcr > =A0 standard form in all oak layers this would allow even > =A0 a broader audience to take advantage of those utilities. +1 to things that are needed in oak-jcr and some of the JCR-specific plugins (name and type validation, etc.) in oak-core. BR, Jukka Zitting