Return-Path: Delivered-To: apmail-incubator-esme-dev-archive@minotaur.apache.org Received: (qmail 28237 invoked from network); 10 Aug 2009 14:08:51 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 10 Aug 2009 14:08:51 -0000 Received: (qmail 26997 invoked by uid 500); 10 Aug 2009 14:08:58 -0000 Delivered-To: apmail-incubator-esme-dev-archive@incubator.apache.org Received: (qmail 26952 invoked by uid 500); 10 Aug 2009 14:08:58 -0000 Mailing-List: contact esme-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: esme-dev@incubator.apache.org Delivered-To: mailing list esme-dev@incubator.apache.org Received: (qmail 26942 invoked by uid 99); 10 Aug 2009 14:08:58 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 10 Aug 2009 14:08:58 +0000 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of hirsch.dick@gmail.com designates 209.85.220.221 as permitted sender) Received: from [209.85.220.221] (HELO mail-fx0-f221.google.com) (209.85.220.221) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 10 Aug 2009 14:08:48 +0000 Received: by fxm21 with SMTP id 21so657687fxm.27 for ; Mon, 10 Aug 2009 07:08:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=oeTxiXyfIsfwhqF3Bpb3ztJfwnd79xoGqLz3zAl498k=; b=UwWw+64z6srYCEkBsPTNHwro0Jv+zJ2dxE7vYeMfH26HtbMte612wqFDUMRXaO/t2r ebtS2hbrBxqIjju40kt7oQ43O4ZvtQDWpX0x6/ynxUPqpN62X71SPvNrmlPzpTKG2BqY QKO5KBPI0NKBk3TqPUbZc3wgPHO7ajux4Y7cE= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=wHiIkmty8NXfDnSmI3oc0U/ut4z84jzM+L1wI8wz186a7t/4VLginMYsZW7ean9R+k qqml7NxOA/CocF4o2eqmqi7IOEx6vRN8f4s7TBmy+vTdDe3XCWWEsjZfBC4R8VHxY+Yz 9lQmjt1sTNURbLCV4xrWW0/jKoGfZP5Pn1ihg= MIME-Version: 1.0 Received: by 10.102.244.4 with SMTP id r4mr400737muh.1.1249913306836; Mon, 10 Aug 2009 07:08:26 -0700 (PDT) In-Reply-To: References: <8d32b3490908031212j6f80db18o7615f64c199b935a@mail.gmail.com> <8d32b3490908051211g19c5e6a6xc1ae0a57972fcb8d@mail.gmail.com> <8d32b3490908061431g7ad6c1b0r7ea55733c0fd5db0@mail.gmail.com> Date: Mon, 10 Aug 2009 16:08:26 +0200 Message-ID: Subject: Re: Stats code From: Richard Hirsch To: esme-dev@incubator.apache.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org I'd like to try out Andy's code but haven't figured out how do it without editing my way to insanity. Does someone else have some sort of a tool (SVN-related??) that takes a svn diff file and edits the source files to contain the changes? D. On Fri, Aug 7, 2009 at 11:56 AM, Richard Hirsch wrote: > Thanks. I'll try and create a deployment based on the patch and deploy > it to Stax on Monday. > > D. > > On Thu, Aug 6, 2009 at 11:31 PM, Andy the > destroyer wrote: >> I uploaded the patch and modified scala-stats.jar to JIRA ( >> https://issues.apache.org/jira/browse/ESME-18). >> >> -Andy >> >> On Wed, Aug 5, 2009 at 12:11 PM, Andy the destroyer < >> andythedestroyer@gmail.com> wrote: >> >>> Hi Vassil, >>> >>> If you look at the scala-stats library, you need to look at my fork of it >>> on my git-hub account (http://github.com/andythedestroyer). I added some >>> more stats and JMX functionality that ESME is using. I provided a link note >>> in the pom section of the patch file. I am in contact with stevej it looks >>> like he is going to merge the changes I made into his master branch; I just >>> need to finish two more features and write the tests for them. >>> >>> I am also working on some actors for the scala-stats lib. I would be >>> interested to see your approach regarding stats and actors. >>> >>> I agree with you that retrieving stats with messages is a good idea. I even >>> started to look at what I would need to do to implement it. However, going >>> through the MsgParser was a nightmare. I am relatively new to Scala and very >>> new to parser combinators. With no comments in the code I found it very hard >>> to navigate my way through it and I moved on to other things. >>> >>> As far as licensing goes, scala-stats is under apache license and aside >>> from specs only has one dependecy-- configgy, which is also under Apache and >>> it's only dependecy is MarkdownJ which is BSD. Actually, the configgy dep is >>> optional. We don't even need it to run with ESME. Although, it might be >>> worth looking at configgy to manage ESME configs. Just a thought.. >>> >>> I will submit the patch to the Jira ticket. >>> >>> Keep the faith, >>> Andy >>> >>> >>> >>> On Wed, Aug 5, 2009 at 4:42 AM, Vassil Dichev wrote: >>> >>>> Thanks Andy, this looks like an efficient use of an external library. >>>> I just heard of stevej's library because it's used in Jonas Boner's >>>> new Akka actors library. I need to take a look into scala-stats and >>>> see what else we can use from this library. >>>> >>>> Incidentally, I'm also writing an actor for popularity stats- e.g. >>>> which messages have been resent the most in the last time period, or >>>> which links have been clicked the most, or which users have been using >>>> a certain tag/word, etc. The statistics from this actor should be >>>> available in a UI snippet for the user to see, but I see no reason >>>> that it shouldn't also be available via JMX. >>>> >>>> As an aside, we could also make an action in a way that we can emulate >>>> a JMX console bot within ESME. For instance, you write a message with >>>> text "stats usersLoggedIn" and you get back a message with the user >>>> count. >>>> >>>> I guess after trying the patch we need to check the licenses of the >>>> used scala-stats library *and* its dependencies (if any). I am not >>>> sure what the exact procedure is for accepting a patch- I think we >>>> need a JIRA item with an attached patch where you click a license >>>> agreement. >>>> >>>> Thanks again, it's nice to see find ESME fun to work on! >>>> >>>> Vassil >>>> >>> >>> >> >