Return-Path: X-Original-To: apmail-devicemap-dev-archive@www.apache.org Delivered-To: apmail-devicemap-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 3C574C2E9 for ; Wed, 7 Jan 2015 17:58:20 +0000 (UTC) Received: (qmail 92990 invoked by uid 500); 7 Jan 2015 17:58:21 -0000 Delivered-To: apmail-devicemap-dev-archive@devicemap.apache.org Received: (qmail 92947 invoked by uid 500); 7 Jan 2015 17:58:21 -0000 Mailing-List: contact dev-help@devicemap.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@devicemap.apache.org Delivered-To: mailing list dev@devicemap.apache.org Received: (qmail 92935 invoked by uid 99); 7 Jan 2015 17:58:20 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 07 Jan 2015 17:58:20 +0000 X-ASF-Spam-Status: No, hits=3.2 required=5.0 tests=FORGED_YAHOO_RCVD,HTML_MESSAGE,RCVD_IN_DNSWL_NONE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of rezmang@yahoo.com designates 72.30.239.77 as permitted sender) Received: from [72.30.239.77] (HELO nm34-vm5.bullet.mail.bf1.yahoo.com) (72.30.239.77) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 07 Jan 2015 17:57:53 +0000 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=s2048; d=yahoo.com; b=Kt2vK0ddb2rjama5txOaobk0asRIsAKQRBZEM0hpFp9jIua6H0gDu4JaKvz9qWe2OLVZQje6nvvplrO7Qh7ut+EUTSMzKfbzv6akxtvTgZBUjkwmf8KJM7OmRYttL9dPKycBz/3xmgW4rPpeBOm7jjDnlJEhFl5Rs9KMBkRMyJbWzhIz1UcnoB79DnoUXV6FIIGzMupGmiu0Ahaj2qXSzecSLawiFcJGecGskjPTtzOv28opD+aQfuE5H3eGFvDhAp8SZ0rYoGXiiBQiB6dy1COdQcJvSvAWWfm+g+72z/HhIML2+9kJyPlu0D1h70yyksPeaClu1C0d3RPQsZiXNQ==; Received: from [98.139.215.142] by nm34.bullet.mail.bf1.yahoo.com with NNFMP; 07 Jan 2015 17:56:48 -0000 Received: from [98.139.212.209] by tm13.bullet.mail.bf1.yahoo.com with NNFMP; 07 Jan 2015 17:56:48 -0000 Received: from [127.0.0.1] by omp1018.mail.bf1.yahoo.com with NNFMP; 07 Jan 2015 17:56:48 -0000 X-Yahoo-Newman-Property: ymail-3 X-Yahoo-Newman-Id: 604286.45489.bm@omp1018.mail.bf1.yahoo.com X-YMail-OSG: CC.PSd0VM1muZoRSYjP8dMzRR4chwLdUgM5afBxzCY3I1ZRRq3UUsinVRvl5m.7 V_lruyxk8ZXjnfe6.2hw2zKFzYvmnLLUTSHcizshrqYPI5pVh639LijT0wLh5hMtdAxpTkrv9WQC 59l6uVQ4lK9Cla5X3wJklPfvi4FW5gje6MwmPYf0aqOqFSbxTcRh70tlaSS9okVvtApbo6lRrrNC uUwxrZf7p5PmCGe4Kxqk_6nDigY0_4trFsnYmwxgoMq.dBJPOAwkmsw7r4zOxm6n.2I9vBGfwfJ2 wZWtbaoBTIAkPVcQhDTET4masHfqEMJ7XbWtWJ1b0njk7beNJREVDwksEwl_rr3copQ.eS4RNrEv zhiBlSqqa46VQqo0EWm_tBdK6PzS7aR_ZB6dFOM1wMriyq3b3hFt3WB0hWVSY1NWpZl8iqzSFU_9 CDfdw83YONSp2u2sO42qHbM_peb1KTAYQf12lgK6ly.kIjNZzYzVyKDwz5gkSUqPiOfb8gMTJdhE W6gbedJ6NPKfaotVzU7RsA83ZeAOyjCCBKIaBqtC9ZoK4FfQ72PTyvDaUMV_65X.h7hUs86ubEVI EaYra4R4xISP04nVkM6ZPtYwGt2INvUZJUhl3zrNMrEo4BFnvcWAmsgdxhx0RZUVhCH3TrSJLtpU jxqMZ_gSXAx7n6VtmIA13amDE3JDU35o- Received: by 76.13.26.71; Wed, 07 Jan 2015 17:56:48 +0000 Date: Wed, 7 Jan 2015 17:56:46 +0000 (UTC) From: Reza Naghibi Reply-To: Reza Naghibi To: "dev@devicemap.apache.org" Message-ID: <20430441.5994099.1420653406513.JavaMail.yahoo@jws10623.mail.bf1.yahoo.com> In-Reply-To: <448819323.5950131.1420652746095.JavaMail.yahoo@jws106104.mail.bf1.yahoo.com> References: <448819323.5950131.1420652746095.JavaMail.yahoo@jws106104.mail.bf1.yahoo.com> Subject: Re: [DISCUSS] Logging in DeviceMap MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_5994098_128975288.1420653406500" X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_5994098_128975288.1420653406500 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Oops, sorry, I think I totally did just minimize this :) But my other point= s still stand... not broken, dont fix, feature creep, bloat. --- From: Reza Naghibi To: "dev@devicemap.apache.org" =20 Sent: Wednesday, January 7, 2015 12:45 PM Subject: Re: [DISCUSS] Logging in DeviceMap =20 I just want to point out that this discussion is a bit of a bikeshed [0]. I= am in no way trying to minimize this or end discussion, but my usual stanc= e is if it isnt broken, dont fix it. Something I want to avoid is bloat and= feature creep. [0]=C2=A0http://bikeshed.com/ =C2=A0 =C2=A0 =C2=A0 From: Werner Keil To: dev@devicemap.apache.org=20 Sent: Wednesday, January 7, 2015 12:01 PM Subject: Re: [DISCUSS] Logging in DeviceMap =C2=A0=20 That's a valid point, which is why ideally they might be consistent. If they're consistent to be all JUL, that might be better than 2 or 3 different ones. The classifier EXAMPLES devicemap/examples (at least dmap-servlet and dmap-spring) point to Log4J 1.x in the POM and if it's not just for Spring or other libraries I guess the examples also use it. Some other examples used either Log4J 1 or SLF4J, too which I unified to be Log4J 2. Ideally all examples could use the same logging API and even better if the clients did, too. Werner On Wed, Jan 7, 2015 at 5:38 PM, Volkan Yaz=C4=B1c=C4=B1 wrote: > Deficiency: Say I have an application X that employs devicemap-client and > uses Log4J for logging. Now the application owner needs to configure > logging for both Log4J and JUL. > > The driver is that: As a library we should not make any assumptions on th= e > used logging framework implementation. > > On Wed, Jan 7, 2015 at 5:10 PM, Reza Naghibi > > wrote: > > > Ok, you made some points. I still have a few questions: > > > > Can you explain some of the known deficiencies for java.util.logging? > > > > Also, again, what is the requirement driving this where the user needs = to > > have better logging configuration and access? > > > >=C2=A0 =C2=A0 =C2=A0 From: Volkan Yaz=C4=B1c=C4=B1 > >=C2=A0 To: "dev@devicemap.apache.org" ; Reza > Naghibi < > > reza.naghibi@yahoo.com> > >=C2=A0 Sent: Wednesday, January 7, 2015 10:53 AM > >=C2=A0 Subject: Re: [DISCUSS] Logging in DeviceMap > > > > My answers are inline. > > > > On Wed, Jan 7, 2015 at 3:49 PM, Reza Naghibi > > > > wrote: > > > > > Im going to say -1 unless there is something specific driving this. I > > dont > > > see anything below other than "Project X did a similar vote". So plea= se > > > reply with reasons, requirements, concerns, etc. > > > > > > > Correct. > > > > My concerns are: > > > > > > -Its another 3rd party library dependency for a non core function. > > Logging > > > is not needed for the client to function properly. Im not against all > 3rd > > > party dependencies. I use them all the time. But there needs to be > solid > > > justification. > > > > > > > Can you please explain your reasons/requirements/concerns for using a > > library that has long been known for its deficiencies and avoid using o= ne > > if its successors? > > > > The best part that I like about SLF4J is that it is a facade, not a > > concrete implementation, and does not try to be like one. I particularl= y > > believe that Apache DeviceMap also should not expose a concrete logger > and > > let the application developer make his decision on how to log things. > That > > is, you can use java.util.logging, Log4J, Logback, etc. with SLF4J. > > > > > > > -Users dont need logging and shouldnt have logging enabled. Its a > > > performance killer. Also, there is nothing in that log stream that ca= n > be > > > helpful to a user. If a user wants to goto the extreme and turn on > > logging, > > > the current logger can facilitate that. > > > > > > > I totally agree, that is why we need to use a facade and let the user > make > > the decision to pick a concrete logger implementation to use. > > > > > > > -I would avoid forcing a logging framework on the project which uses > this > > > library. > > > > > > > Using JUL already forces one. SLF4J will give a freedom of choice. > > > > > > > > > > > So I would look at how Spring does logging [0]. If anything, maybe > detect > > > a logger and use it. But once again, thats going to be a performance > hit > > > and I think some of my points will still hold regardless of the > approach. > > > > > > [0] > > > > > > http://docs.spring.io/spring-boot/docs/current/reference/html/howto-loggi= ng.html > > > > > > Yes, we should check what Spring does. An excerpt from Spring Framework > > Reference Documentation > > < > > > http://docs.spring.io/spring/docs/current/spring-framework-reference/html= single/#overview-logging > > > > > : > > > > Unfortunately, the runtime discovery algorithm in commons-logging, whil= e > > convenient for the end-user, is problematic. If we could turn back the > > clock and start Spring now as a new project it would use a different > > logging dependency. The first choice would probably be the Simple Loggi= ng > > Facade for Java ( SLF4J), which is also used by a lot of other tools th= at > > people use with Spring inside their applications. > > > > > > Best. > > > > > > > > > ------=_Part_5994098_128975288.1420653406500--