Return-Path: X-Original-To: apmail-commons-dev-archive@www.apache.org Delivered-To: apmail-commons-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 6BE411136C for ; Mon, 2 Jun 2014 15:03:25 +0000 (UTC) Received: (qmail 63036 invoked by uid 500); 2 Jun 2014 15:03:25 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 62896 invoked by uid 500); 2 Jun 2014 15:03:24 -0000 Mailing-List: contact dev-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Commons Developers List" Delivered-To: mailing list dev@commons.apache.org Received: (qmail 62885 invoked by uid 99); 2 Jun 2014 15:03:24 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 02 Jun 2014 15:03:24 +0000 X-ASF-Spam-Status: No, hits=2.2 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_NONE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of rmannibucau@gmail.com designates 209.85.192.43 as permitted sender) Received: from [209.85.192.43] (HELO mail-qg0-f43.google.com) (209.85.192.43) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 02 Jun 2014 15:03:21 +0000 Received: by mail-qg0-f43.google.com with SMTP id 63so10895421qgz.2 for ; Mon, 02 Jun 2014 08:03:00 -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; bh=q4iaWRgiAJrKYic8eL2xSeF/ZbVZmqvHNEiSU5gSdKY=; b=kXPB9bBSE8xpUlgqPut0ExCrkFdg0jtd6zzsyy7MeKnqEjZjlB/r6u1AfPbNB86d9Y uJsEFvawzSMQsGw9QcHitUzw2DXa1SGhLYVjyxc/+yRw7RsMSb5OEa+kbVS8KN1Bczna sdo5uIc6duxWIhbtmdVftvRX66nQeQEPR3I8xafZFoprXa0Gwrytc6dz1kn26o91DB5P SN6lQhgd/LKhDl06J0D7I/RAdWOIEbPftoaMEfFRDMnjo1DvE2w/srf0jl2XPn5fDQ9V Ws0+Gg6JkShiO0RRDhh2QxPKaiUVsH3XnA8cMoL8KygaT9Igmo4LbHejyAK2ThA/UNls t7mQ== X-Received: by 10.229.220.197 with SMTP id hz5mr49655332qcb.9.1401721380283; Mon, 02 Jun 2014 08:03:00 -0700 (PDT) MIME-Version: 1.0 Received: by 10.96.123.7 with HTTP; Mon, 2 Jun 2014 08:02:40 -0700 (PDT) In-Reply-To: <538C8820.3070206@gmail.com> References: <538C8820.3070206@gmail.com> From: Romain Manni-Bucau Date: Mon, 2 Jun 2014 17:02:40 +0200 Message-ID: Subject: Re: [jcs] logging To: Commons Developers List Content-Type: multipart/alternative; boundary=001a11346f6af22cc804fadbb00a X-Virus-Checked: Checked by ClamAV on apache.org --001a11346f6af22cc804fadbb00a Content-Type: text/plain; charset=UTF-8 actually these logs are really debug log not intended to be activated in prod excepted if there is a big issue (@thomas: if you can confirm it would be great). Romain Manni-Bucau Twitter: @rmannibucau Blog: http://rmannibucau.wordpress.com/ LinkedIn: http://fr.linkedin.com/in/rmannibucau Github: https://github.com/rmannibucau 2014-06-02 16:20 GMT+02:00 Phil Steitz : > On 6/1/14, 12:26 PM, Romain Manni-Bucau wrote: > > Hi > > > > I have two main point to discuss regarding the logging: > > 1) LogHelper stuffI committed. Idea was to cache isDebugEnabled to get a > if > > (boolean) complexity and not go through the logging framework which can > > imply several layers (filter, appender, handler, logger...) for nothing > and > > slow down caching (which has more perf constraints than other backends. > > This really depends on the logger you use but we can't suppose it is the > > one we benched against. Solutions I see: a) keep LogHelper, b) remove > logs > > (some are useless I think), 3) other? > > Let me ask the obvious, probably naive question: does this component > really need logging at all? Could you expose what clients need via > JMX or other means? It would be good to know what clients are > actually using / depending on. > > Phil > > > > 2) logger api used. ATM we use [logging] but it will surely be an issue > for > > TomEE when integrated ([logging] is the less integrated framework - > > compared to JUL or SLF4J where we don't have the choice at all) and I > think > > we'd be happy to remove it from the container to let it be application > > oriented if we can. Any idea to make it hurtless? This doesn't urge and > > doesn't block anything but if someone has an awesome idea it would be > > welcomed ;) > > > > > > Romain Manni-Bucau > > Twitter: @rmannibucau > > Blog: http://rmannibucau.wordpress.com/ > > LinkedIn: http://fr.linkedin.com/in/rmannibucau > > Github: https://github.com/rmannibucau > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org > For additional commands, e-mail: dev-help@commons.apache.org > > --001a11346f6af22cc804fadbb00a--