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 007B59A10 for ; Wed, 7 Mar 2012 19:31:54 +0000 (UTC) Received: (qmail 22353 invoked by uid 500); 7 Mar 2012 19:31:54 -0000 Delivered-To: apmail-commons-dev-archive@commons.apache.org Received: (qmail 22196 invoked by uid 500); 7 Mar 2012 19:31:54 -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 22182 invoked by uid 99); 7 Mar 2012 19:31:54 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 07 Mar 2012 19:31:54 +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 (nike.apache.org: domain of simone.tripodi@gmail.com designates 209.85.216.171 as permitted sender) Received: from [209.85.216.171] (HELO mail-qy0-f171.google.com) (209.85.216.171) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 07 Mar 2012 19:31:48 +0000 Received: by qcsp15 with SMTP id p15so4311171qcs.30 for ; Wed, 07 Mar 2012 11:31:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type :content-transfer-encoding; bh=oKbMOELgUM3ohDKHgA/mwusae3TI3hJzRi3Mkncdnsk=; b=NPuvkC1/sypUQXlLG4jiGiqwQ01h7qyiTCH37ssdh+i3n9iA9TiPf3hEVwSVkoxlZC hxz2Ujdid+HmGQlvGXW8IM/r4YneJa3uKWnG+fMbtDhIASE4lV9bPEv23fKlS1SWZMAE hoWVF6K3zA9dwVSkrc4xxVNvPR1ccrz6D2NHR5X/sfwkdp500TH4UdyOr4JHV02X1W9Y 3ncT81r80wA6f6cgroT/jP0sn4j9HeGY9A4/fBWzUYH7Cw7iDNMrrYn7iHFmeLY9sdMm Jz52DeMHYT0UOdFwoThTjzhkiYvoXkbypHRMOK9YZIsIo5Z3Kwu4aZCktNzOxfReJDa4 1nHA== MIME-Version: 1.0 Received: by 10.229.114.222 with SMTP id f30mr818480qcq.13.1331148687746; Wed, 07 Mar 2012 11:31:27 -0800 (PST) Sender: simone.tripodi@gmail.com Received: by 10.224.195.73 with HTTP; Wed, 7 Mar 2012 11:31:27 -0800 (PST) In-Reply-To: References: <4F564C74.7000303@apache.org> <4F575015.5010306@apache.org> Date: Wed, 7 Mar 2012 20:31:27 +0100 X-Google-Sender-Auth: HdwjYk-3AEdXUy2i1G9sogRexsY Message-ID: Subject: Re: [VOTE] Promote [csv] to Commons proper From: Simone Tripodi To: Commons Developers List Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Virus-Checked: Checked by ClamAV on apache.org Hi all, IMHO if you could implement the readers as XMLReader instances, you can get benefit from the powerful Digester matching rules and POJOs mapping. I had a spike with Digester reading JSON via a javacc grammar but it was not complete so never proposed it. just my 0.0000002 cents, -Simo http://people.apache.org/~simonetripodi/ http://simonetripodi.livejournal.com/ http://twitter.com/simonetripodi http://www.99soft.org/ On Wed, Mar 7, 2012 at 8:11 PM, Gary Gregory wrote= : > On Wed, Mar 7, 2012 at 7:09 AM, Emmanuel Bourg wrote: > >> Le 07/03/2012 12:57, sebb a =C3=A9crit : >> >> >> =C2=A0Since CSV is currently only a single package with very few classes= , >>> would it perhaps be suitable as a part of an existing Commons >>> component? >>> >> >> [csv] is still small but will probably increase in size as more features >> are integrated (like the bean mapping). I prefer to leave it as an >> independent component. If it was to be merged with another component in = the >> future I think [flatfile] would be a better candidate. > > > I like the name [flatfile], it does imply something more generic than jus= t > *Comma* separated values. > > I am intrigued by the idea of merging this into an existing component but= I > do not see which one would really fit. [io] comes close but I my mind > [flatfile] could evolve into a JDBC driver. When I think about it that wa= y, > it does not fit into [io]. > > I do like the idea of merging certain things though, so I might bring up > merging [exec] into [lang] for example... > > Gary > > > >> >> >> =C2=A0[This would solve the package name issue.] >>> >> >> Let's not try contortions to solve this issue. We have to admit that the >> impact is limited. The solr-commons-csv artifact is not widely used, Sol= r >> is going to fix the next releases, and for people importing the previous >> release it'll be possible to exclude the dependency to avoid a conflict. >> >> It's not perfect but it's good enough to keep the current package and >> class names unchanged. Well put a warning on the main page to document t= he >> issue. >> >> Emmanuel Bourg >> >> > > > -- > E-Mail: garydgregory@gmail.com | ggregory@apache.org > JUnit in Action, 2nd Ed: http://bit.ly/ECvg0 > Spring Batch in Action: http://bit.ly/bqpbCK > Blog: http://garygregory.wordpress.com > Home: http://garygregory.com/ > Tweet! http://twitter.com/GaryGregory --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org For additional commands, e-mail: dev-help@commons.apache.org