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 014D610EBB for ; Wed, 26 Mar 2014 10:10:15 +0000 (UTC) Received: (qmail 41382 invoked by uid 500); 26 Mar 2014 10:10:14 -0000 Delivered-To: apmail-jackrabbit-oak-dev-archive@jackrabbit.apache.org Received: (qmail 41214 invoked by uid 500); 26 Mar 2014 10:10:10 -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 41191 invoked by uid 99); 26 Mar 2014 10:10:08 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 26 Mar 2014 10:10:08 +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 chetan.mehrotra@gmail.com designates 209.85.214.175 as permitted sender) Received: from [209.85.214.175] (HELO mail-ob0-f175.google.com) (209.85.214.175) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 26 Mar 2014 10:10:03 +0000 Received: by mail-ob0-f175.google.com with SMTP id uy5so2191141obc.20 for ; Wed, 26 Mar 2014 03:09:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; bh=1pndFgtzar0F0HK2vLyYdSonvK70958Ytiuv46d1MB0=; b=xk+hoYI03FpdSDf3DwsGdXGvdiHH59znDHl9Wb/ydya7IuJN8XlUb2vtknFMgcGIRD hkgA1QmtF/xpnPl54KUP0P+r7mL7zyGjMjcDQI9pixQtTVFIrSXMWg3T5z4oPsYw+RBZ Y959U33lJTE3D8tm1nTDBBySWDv/ta3DV3SoNMh54dEzpAIYxVxXmFWfa95b/gLAjBGC kRceHO6WTWHoCK15jH1Uo6/oAzoPCzHf4ppuUb0G4ef2K/N2D09V6Iz/s4GXuEh8nq5c Ql455+iuLHudUN065J68Eye8h6PVDcbFlDZPAzANexZhJeFaZ9UAkbrLZCiwjE/odqPu +6Xg== MIME-Version: 1.0 X-Received: by 10.182.241.9 with SMTP id we9mr13420obc.81.1395828582425; Wed, 26 Mar 2014 03:09:42 -0700 (PDT) Received: by 10.60.93.163 with HTTP; Wed, 26 Mar 2014 03:09:42 -0700 (PDT) In-Reply-To: References: Date: Wed, 26 Mar 2014 15:39:42 +0530 Message-ID: Subject: Re: Request for feedback: OSGi Configuration for Query Limits (OAK-1571) From: Chetan Mehrotra 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 Patch looks fine to me. Probably we can collapse QueryIndexProvider and QueryEngineSettings into a single QueryEngineContext and pass that along till Root. > So: is it worth it to have the 100 KB source code overhead just to make t= hings configurable separately for each Oak instance? I think there are couple of benefits * Isolation between multiple oak instance running on same jvm (minor) * It opens up possibility to have session specific settings. So later we require say JR2 compatible behaviour case for some session then those settings can be overlayed with session attributes * it allows to change the setting at runtime via gui as some of these settings would not require repository restart and can effect the next query that gets executed. That would be a major win So this effort now would enable incremental improvements in QueryEngine in future! > The Whiteboard is per Oak instance, right? For OSGi case yes Chetan Mehrotra On Wed, Mar 26, 2014 at 2:23 PM, Thomas Mueller wrote: > Hi, > > I'm trying to make some query settings (limits on the number of nodes rea= d) configurable via OSGi. So far, I have a patch of about 100 KB, and this = is just wiring together the components (no OSGi / Whiteboard so far). > > I wonder, is there an easier way to do it? With system properties, it's j= ust a few lines of code. The disadvantage is that all Oak instances in the = same JVM use the same settings, but with OSGi configuration I guess in real= ity it's not much different. So: is it worth it to have the 100 KB source c= ode overhead just to make things configurable separately for each Oak insta= nce? If not, how could it be implemented? The Whiteboard is per Oak instanc= e, right? > > Regards, > Thomas > >