Re: [text] Deprecate CsvTranslators

[Amey Jadiye <am...@gmail.com>]

well, i'm still unclear about the action to be taken here, so i'll keep on
beating a dead horse till we fix it or close it at some logic end.

Regards,
Amey

On Fri, May 26, 2017 at 12:14 PM, Amey Jadiye <am...@gmail.com> wrote:

> Hi Gary, just to conclude the final action and clear midst here, I wish
> you and team to elaborate your thought more on.
>
> 1. Deprecate CsvTranslator and point javadoc to use same methods present
> in StringEscapeUtils in Commons Text.(moved from lang3)
>
> 2. Move CsvTranalator to Commons CSV all together.
>
> 3. Anything else ?
>
> I like to go with 1st point which I agreed upon before.
>
> Regards,
> Amey
>
>
> On Fri, May 26, 2017, 12:41 AM Gary Gregory <ga...@gmail.com>
> wrote:
>
>> Mentioning the big picture again, Javadocs could also point to Commons
>> CSV.
>>
>> Gary
>>
>> On May 25, 2017 12:06 PM, "Amey Jadiye" <am...@gmail.com> wrote:
>>
>> > Agreed!, Thanks for sharing thoughts gary, now only marking
>> CsvTranslator
>> > as deprecated will be OK.
>> >
>> > I'm marking methods from commons-text StringEscapeUtils as alternative
>> > to CsvTranslator,
>> > Thanks.
>> >
>> > Regards,
>> > Amey
>> >
>> > On Fri, May 26, 2017 at 12:25 AM, Gary Gregory <ga...@gmail.com>
>> > wrote:
>> >
>> > > I think we need to step back here, or up, and look at the big picture.
>> > >
>> > > Commons CSV reads and writes CSV files. There a couple of additional
>> > cases
>> > > like writing CSV from JDBC result set.
>> > >
>> > > The Commons Text CSV methods do low level work that is already done
>> > within
>> > > the guts of Commons CSV.
>> > >
>> > > What is the use case of Commons Text and Lang CSV methods? Build
>> parts of
>> > > CSV IO code I would guess.
>> > >
>> > > I do not want to just move code between components in this case. This
>> is
>> > > not like WordUtils moving from Lang to Text.
>> > >
>> > > How is this code used? Can a user just do away with it by recoding to
>> > > Commons CSV?
>> > >
>> > > Gary
>> > >
>> > > On May 25, 2017 11:26 AM, "Amey Jadiye" <am...@gmail.com> wrote:
>> > >
>> > > > Hi Gary,
>> > > >
>> > > > I have deprecated the CsvTranslator from commons text and was
>> looking
>> > for
>> > > > alternate API, we have it in StringEscapeUtils in commons text (and
>> in
>> > > > commons lang3 which is deprecated & which i'm not worry about), I
>> think
>> > > we
>> > > > should remove those csv related methods from StringEscapeUtils as
>> well
>> > > and
>> > > > create CsvEscapeUtils in commons CSV as its better place, thoughts ?
>> > > >
>> > > > Regards,
>> > > > Amey
>> > > >
>> > > >
>> > > >
>> > > > On Thu, May 25, 2017 at 1:04 AM, Gary Gregory <
>> garydgregory@gmail.com>
>> > > > wrote:
>> > > >
>> > > > > I'm not talking about moving the [text] classes to [csv] but
>> rather I
>> > > > want
>> > > > > folks to use [csv] instead of [txt]. If we have to add a few APIs
>> > here
>> > > > and
>> > > > > there fine. But I would hope that all we need is there already.
>> > > > >
>> > > > > Gary
>> > > > >
>> > > > > On Tue, May 23, 2017 at 3:24 AM, Emmanuel Bourg <
>> ebourg@apache.org>
>> > > > wrote:
>> > > > >
>> > > > > > Le 23/05/2017 à 01:06, Gary Gregory a écrit :
>> > > > > >
>> > > > > > > I wonder if we should deprecate CsvTranslators in favor of
>> > Commons
>> > > > > CSV. I
>> > > > > > > wish I had noticed CsvTranslators eariler :-(
>> > > > > >
>> > > > > > +1
>> > > > > >
>> > > > > > Emmanuel Bourg
>> > > > > >
>> > > > > >
>> > > > > > ------------------------------------------------------------
>> > > ---------
>> > > > > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> > > > > > For additional commands, e-mail: dev-help@commons.apache.org
>> > > > > >
>> > > > > >
>> > > > >
>> > > > >
>> > > > > --
>> > > > > E-Mail: garydgregory@gmail.com | ggregory@apache.org
>> > > > > Java Persistence with Hibernate, Second Edition
>> > > > > <https://www.amazon.com/gp/product/1617290459/ref=as_li_
>> > > > > tl?ie=UTF8&camp=1789&creative=9325&creativeASIN=1617290459&
>> > > > > linkCode=as2&tag=garygregory-20&linkId=
>> > cadb800f39946ec62ea2b1af9fe6a2
>> > > b8>
>> > > > >
>> > > > > <http:////ir-na.amazon-adsystem.com/e/ir?t=
>> > garygregory-20&l=am2&o=1&a=
>> > > > > 1617290459>
>> > > > > JUnit in Action, Second Edition
>> > > > > <https://www.amazon.com/gp/product/1935182021/ref=as_li_
>> > > > > tl?ie=UTF8&camp=1789&creative=9325&creativeASIN=1935182021&
>> > > > > linkCode=as2&tag=garygregory-20&linkId=
>> > 31ecd1f6b6d1eaf8886ac902a24de4
>> > > > 18%22
>> > > > > >
>> > > > >
>> > > > > <http:////ir-na.amazon-adsystem.com/e/ir?t=
>> > garygregory-20&l=am2&o=1&a=
>> > > > > 1935182021>
>> > > > > Spring Batch in Action
>> > > > > <https://www.amazon.com/gp/product/1935182951/ref=as_li_
>> > > > > tl?ie=UTF8&camp=1789&creative=9325&creativeASIN=1935182951&
>> > > > > linkCode=%7B%7BlinkCode%7D%7D&tag=garygregory-20&linkId=%7B%
>> > > > > 7Blink_id%7D%7D%22%3ESpring+Batch+in+Action>
>> > > > > <http:////ir-na.amazon-adsystem.com/e/ir?t=
>> > garygregory-20&l=am2&o=1&a=
>> > > > > 1935182951>
>> > > > > 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
>> > > >
>> > >
>> >
>> >
>> >
>> > --
>> >
>> > ---------------------------------------------------------------------
>> >
>> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> >
>> > For additional commands, e-mail: dev-help@commons.apache.org
>> >
>>
>


-- 

---------------------------------------------------------------------

To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org

For additional commands, e-mail: dev-help@commons.apache.org