Re: [1/4] [text] Deprecate CsvTranslators from from commons text in favor of Commons CSV

2017-06-12 Thread Rob Tompkins
rough the discussion. The class at the HEAD of master should not have the deprecation in place. Do check me on that though. -Rob > > Gary > > -- Forwarded message -- > From: > Date: Mon, Jun 12, 2017 at 4:41 AM > Subject: [1/4] [text] Deprecate CsvTr

Fwd: [1/4] [text] Deprecate CsvTranslators from from commons text in favor of Commons CSV

2017-06-12 Thread Gary Gregory
1/4] [text] Deprecate CsvTranslators from from commons text in favor of Commons CSV To: comm...@commons.apache.org Repository: commons-text Updated Branches: refs/heads/master 640097555 -> 6a5fcfcba Deprecate CsvTranslators from from commons text in favor of Commons CSV Project: http://git-wip-

Re: [text] Deprecate CsvTranslators

2017-06-10 Thread Amey Jadiye
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 wrote: > Hi Gary, just to conclude the final action and clear midst here, I wish > you and te

Re: [text] Deprecate CsvTranslators

2017-05-25 Thread Amey Jadiye
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 togethe

Re: [text] Deprecate CsvTranslators

2017-05-25 Thread Gary Gregory
Mentioning the big picture again, Javadocs could also point to Commons CSV. Gary On May 25, 2017 12:06 PM, "Amey Jadiye" 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

Re: [text] Deprecate CsvTranslators

2017-05-25 Thread Amey Jadiye
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 wrote: > I think we need to step bac

Re: [text] Deprecate CsvTranslators

2017-05-25 Thread Gary Gregory
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

Re: [text] Deprecate CsvTranslators

2017-05-25 Thread Amey Jadiye
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 wel

Re: [text] Deprecate CsvTranslators

2017-05-24 Thread Amey Jadiye
+1 On May 25, 2017 1:04 AM, "Gary Gregory" 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 2

Re: [text] Deprecate CsvTranslators

2017-05-24 Thread Gary Gregory
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 wrote: > Le 23/05/2017 à 01:

Re: [text] Deprecate CsvTranslators

2017-05-23 Thread Emmanuel Bourg
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-unsubscr

Re: [text] Deprecate CsvTranslators

2017-05-23 Thread Amey Jadiye
Seems reasonable to me as well, should deprecate in text 1.2 release and code should be move to Commons CSV , I think complete removal should go in Text 2.0 , whenever it will be. Regards, Amey On Tue, May 23, 2017, 4:41 AM Rob Tompkins wrote: > > > > On May 22, 2017, at 7:06 PM, Gary Gregory

Re: [text] Deprecate CsvTranslators

2017-05-22 Thread Rob Tompkins
> On May 22, 2017, at 7:06 PM, Gary Gregory wrote: > > Hi All: > > I wonder if we should deprecate CsvTranslators in favor of Commons CSV. I > wish I had noticed CsvTranslators eariler :-( Seems reasonable to me. Curious if there are general arguments against frequent major version releases?

[text] Deprecate CsvTranslators

2017-05-22 Thread Gary Gregory
Hi All: I wonder if we should deprecate CsvTranslators in favor of Commons CSV. I wish I had noticed CsvTranslators eariler :-( Gary -- E-Mail: garydgreg...@gmail.com | ggreg...@apache.org Java Persistence with Hibernate, Second Edition