Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-12 Thread Gary Gregory
Another thing I'll mention: - We are starting to add testing with IBM Java 8 to Travis-CI builds. Gary On Mon, Jun 12, 2017 at 11:28 PM, Gary Gregory wrote: > > On Mon, Jun 12, 2017 at 11:05 PM, Benedikt Ritter > wrote: > >> Hello Gary, >> >> > Am 13.06.2017 um 03:23 schrieb Gary Gregory : >>

Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-12 Thread Gary Gregory
On Mon, Jun 12, 2017 at 11:05 PM, Benedikt Ritter wrote: > Hello Gary, > > > Am 13.06.2017 um 03:23 schrieb Gary Gregory : > > > > Any and all feedback is welcome. I would like to send the report in 24 > > hours. > > > > ## Description: > > > > - The Apache Commons project focuses on all aspects

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121588869 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589195 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589022 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589195 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121589022 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121588869 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,15 @@ public void testURLPattern() throws Exception

Re: [DRAFT] Apache Commons Board Report for June 2017

2017-06-12 Thread Benedikt Ritter
Hello Gary, > Am 13.06.2017 um 03:23 schrieb Gary Gregory : > > Any and all feedback is welcome. I would like to send the report in 24 > hours. > > ## Description: > > - The Apache Commons project focuses on all aspects of reusable Java > components. > > - The Apache Commons components are wid

Re: [6/6] [text] Merge branch 'travis-ibmjdk8' of https://github.com/ameyjadiye/commons-text

2017-06-12 Thread Amey Jadiye
I have created PR for Commons Lang, you may to copy same file to other components. Thanks, Amey On Tue, Jun 13, 2017 at 2:49 AM, Rob Tompkins wrote: > > > > On Jun 12, 2017, at 5:14 PM, Gary Gregory > wrote: > > > > Hi Rob, > > > > Thank you for pushing this through. > > > > Would you consider

Re: Commons sub project for parallel method execution

2017-06-12 Thread Matt Sicker
There's also some interesting execution APIs available in the Scala standard library. Those are built on top of ForkJoinPool and such nowadays, but the idea is there for a nicer API on top of ExecutorService and other low level details. In the interests of concurrency, there are other thread-like

Re: Commons sub project for parallel method execution

2017-06-12 Thread Bruno P. Kinoshita
Interesting idea. And great discussion. Can't really say I'd have a use case for that right now, so abstaining from the discussion around the implementation. I believe if we decide to explore this idea in Commons, we will probably move it to sandbox? Even if we do not move that to Commons or to

Re: Commons sub project for parallel method execution

2017-06-12 Thread Arun Mohan
A lot of useful and interesting suggestions here. Usage of annotations instead of hardcoding method names is definitely a very good idea. Decorating the domain classes with annotations will be much more clean and maintainable. And I think we can check out the usage of MethodHandle's interoperabilit

Re: Commons sub project for parallel method execution

2017-06-12 Thread Bill Igoe
A java CUDA jni library would be excellent. On Jun 12, 2017 7:08 PM, "Gary Gregory" wrote: > On Mon, Jun 12, 2017 at 6:56 PM, Matt Sicker wrote: > > > So wouldn't something like ASM or Javassist or one of the zillion other > > bytecode libraries be a better alternative to using reflection for >

Re: Commons sub project for parallel method execution

2017-06-12 Thread Gary Gregory
On Mon, Jun 12, 2017 at 6:56 PM, Matt Sicker wrote: > So wouldn't something like ASM or Javassist or one of the zillion other > bytecode libraries be a better alternative to using reflection for > performance? Also, using the Java 7 reflections API improvements helps > speed things up quite a bit

Re: Commons sub project for parallel method execution

2017-06-12 Thread Matt Sicker
So wouldn't something like ASM or Javassist or one of the zillion other bytecode libraries be a better alternative to using reflection for performance? Also, using the Java 7 reflections API improvements helps speed things up quite a bit. On 12 June 2017 at 20:37, Paul King wrote: > My goto libr

Re: Commons sub project for parallel method execution

2017-06-12 Thread Paul King
My goto library for such tasks would be GPars. It has both Java and Groovy support for most things (actors/dataflow) but less so for asynchronous task execution. It's one of the things that would be good to explore in light of Java 8. Groovy is now Apache, GPars not at this stage. So with adding t

Re: Commons sub project for parallel method execution

2017-06-12 Thread Gary Gregory
On Mon, Jun 12, 2017 at 6:21 PM, Matt Sicker wrote: > Last time I used Spring, they had an @Async annotation you could use which > would automatically execute in an executor service (all handled via bean > proxies as usual). > Yeah, for higher-level stuff like Spring Batch, it's a little more aw

[DRAFT] Apache Commons Board Report for June 2017

2017-06-12 Thread Gary Gregory
Any and all feedback is welcome. I would like to send the report in 24 hours. ## Description: - The Apache Commons project focuses on all aspects of reusable Java components. - The Apache Commons components are widely used in many projects, both within Apache and without. Any ASF committer can c

Re: Commons sub project for parallel method execution

2017-06-12 Thread Matt Sicker
Last time I used Spring, they had an @Async annotation you could use which would automatically execute in an executor service (all handled via bean proxies as usual). On 12 June 2017 at 19:22, Gary Gregory wrote: > Hi All, > > I think it would be most helpful to note the distinction between the

Re: Commons sub project for parallel method execution

2017-06-12 Thread Gary Gregory
Hi All, I think it would be most helpful to note the distinction between the parallelism aspect and the bridge to domain classes aspect (currently done with reflection in the proposed github repo.) It seems (to me) that in between the ForkJoin framework already in Java (a low-level library) and u

Re: Commons sub project for parallel method execution

2017-06-12 Thread Matt Sicker
I'd be interested to see where this leads to. It could end up as a sort of Commons Parallel library. Besides providing an execution API, there could be plenty of support utilities that tend to be found in all the *Util(s)/*Helper classes in projects like all the ones I mentioned earlier (basically

Re: Commons sub project for parallel method execution

2017-06-12 Thread Gary Gregory
The upshot is that there has to be a way to do this with some custom code to at least have the ability to 'fast path' the code without reflection. Using lambdas should make this fairly syntactically unobtrusive. On Mon, Jun 12, 2017 at 4:02 PM, Arun Mohan wrote: > Yes, reflection is not very per

Re: Commons sub project for parallel method execution

2017-06-12 Thread Arun Mohan
Yes, reflection is not very performant but I don't think I have any other choice since the library has to inspect the object supplied by the client at runtime to pick out the methods to be invoked using CompletableFuture. But the performance penalty paid for using reflection will be more than offse

Re: Commons sub project for parallel method execution

2017-06-12 Thread Gary Gregory
On a lower-level, if you want to use this for lower-level services (where there is no network latency for example), you will need to avoid using reflection to get the best performance. Gary On Mon, Jun 12, 2017 at 3:15 PM, Arun Mohan wrote: > Hi Gary, > > Thanks for your response. You have some

Re: Commons sub project for parallel method execution

2017-06-12 Thread Arun Mohan
Hi Gary, Thanks for your response. You have some valid and interesting points :-) Of course you are right that Spark is much more mature. Thanks for your insight. It will be interesting indeed to find out if the core parallelization engine of Spark can be isolated like you suggest. I started work

Re: Commons sub project for parallel method execution

2017-06-12 Thread Gary Gregory
Hi Arun, Sure, and that is to be expected, Spark is more mature than a four class prototype. What I am trying to get to is that in order for the library to be useful, you will end up with more in a first release, and after a couple more releases, there will be more and more. Would Spark not have i

Re: Commons sub project for parallel method execution

2017-06-12 Thread Arun Mohan
Hi, This project is a pure java project with no dependencies on any library outside the JDK. It can be used with any java based project or java frameworks like Spring as just a jar file. Given some objects and a couple of methods on these objects, the library can execute these methods in parallel.

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-12 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11939016/badge)](https://coveralls.io/builds/11939016) Coverage increased (+0.009%) to 96.244% when pulling **4f9c95bcb246b64f7f675

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-12 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11939016/badge)](https://coveralls.io/builds/11939016) Coverage increased (+0.009%) to 96.244% when pulling **4f9c95bcb246b64f7f675

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-12 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11939016/badge)](https://coveralls.io/builds/11939016) Coverage increased (+0.009%) to 96.244% when pulling **4f9c95bcb246b64f7f675

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-12 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11939016/badge)](https://coveralls.io/builds/11939016) Coverage increased (+0.009%) to 96.244% when pulling **4f9c95bcb246b64f7f675

Re: Commons sub project for parallel method execution

2017-06-12 Thread Matt Sicker
I already see a huge difference here: Spark requires a bunch of infrastructure to be set up, while this library is just a library. Similar to Kafka Streams versus Spark Streaming or Flink or Storm or Samza or the others. On 12 June 2017 at 16:28, Gary Gregory wrote: > On Mon, Jun 12, 2017 at 2:2

Re: [6/6] [text] Merge branch 'travis-ibmjdk8' of https://github.com/ameyjadiye/commons-text

2017-06-12 Thread Gary Gregory
On Mon, Jun 12, 2017 at 2:19 PM, Rob Tompkins wrote: > > > > On Jun 12, 2017, at 5:14 PM, Gary Gregory > wrote: > > > > Hi Rob, > > > > Thank you for pushing this through. > > > > Would you consider doing the same for the few other Travis builds we have > > in Commons? > > Sure. I'll start chipp

Re: Commons sub project for parallel method execution

2017-06-12 Thread Gary Gregory
On Mon, Jun 12, 2017 at 2:26 PM, Arun Mohan wrote: > Hi All, > > Good afternoon. > > I have been working on a java generic parallel execution library which will > allow clients to execute methods in parallel irrespective of the number of > method arguments, type of method arguments, return type o

Commons sub project for parallel method execution

2017-06-12 Thread Arun Mohan
Hi All, Good afternoon. I have been working on a java generic parallel execution library which will allow clients to execute methods in parallel irrespective of the number of method arguments, type of method arguments, return type of the method etc. Here is the link to the source code: https://g

Re: [6/6] [text] Merge branch 'travis-ibmjdk8' of https://github.com/ameyjadiye/commons-text

2017-06-12 Thread Rob Tompkins
> On Jun 12, 2017, at 5:14 PM, Gary Gregory wrote: > > Hi Rob, > > Thank you for pushing this through. > > Would you consider doing the same for the few other Travis builds we have > in Commons? Sure. I'll start chipping away at that. -Rob > > Thank you, > Gary > >> -- Forwarded

Fwd: [6/6] [text] Merge branch 'travis-ibmjdk8' of https://github.com/ameyjadiye/commons-text

2017-06-12 Thread Gary Gregory
Hi Rob, Thank you for pushing this through. Would you consider doing the same for the few other Travis builds we have in Commons? Thank you, Gary > -- Forwarded message -- > From: > Date: Mon, Jun 12, 2017 at 4:58 AM > Subject: [6/6] [text] Merge branch 'travis-ibmjdk8' of > ht

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-12 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11938473/badge)](https://coveralls.io/builds/11938473) Coverage increased (+0.009%) to 96.244% when pulling **1e59d0c2fd1cfee450d01

[GitHub] commons-cli issue #12: [CLI-274] implement EXISTING_FILE_VALUE type handler

2017-06-12 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/12 [![Coverage Status](https://coveralls.io/builds/11938473/badge)](https://coveralls.io/builds/11938473) Coverage increased (+0.009%) to 96.244% when pulling **1e59d0c2fd1cfee450d01

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

2017-06-12 Thread Rob Tompkins
> On Jun 12, 2017, at 4:37 PM, Gary Gregory wrote: > > IMO it is not enough to @Deprecated a whole class. You need a @Deprecated > in the Javadoc with an explanation of what to replace it with. Like "Use > Foo#bar() in Apache Commons CSV". That was a commit mid way through the discussion. The

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

2017-06-12 Thread Gary Gregory
IMO it is not enough to @Deprecated a whole class. You need a @Deprecated in the Javadoc with an explanation of what to replace it with. Like "Use Foo#bar() in Apache Commons CSV". Gary -- Forwarded message -- From: Date: Mon, Jun 12, 2017 at 4:41 AM Subject: [1/4] [text] Depreca

[GitHub] commons-collections pull request #23: COLLECTIONS-606:Added build and covera...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-collections/pull/23 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121493262 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,12 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread britter
Github user britter commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121493262 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,12 @@ public void testURLPattern() throws Exception

[GitHub] commons-compress issue #31: COMPRESS-400 It should be possible for users to ...

2017-06-12 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-compress/pull/31 [![Coverage Status](https://coveralls.io/builds/11935842/badge)](https://coveralls.io/builds/11935842) Coverage decreased (-0.03%) to 84.306% when pulling **3a1d29e2474519c39

[GitHub] commons-compress pull request #27: COMPRESS-400

2017-06-12 Thread sesuncedu
Github user sesuncedu closed the pull request at: https://github.com/apache/commons-compress/pull/27 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fea

[GitHub] commons-compress pull request #31: COMPRESS-400 It should be possible for us...

2017-06-12 Thread sesuncedu
GitHub user sesuncedu opened a pull request: https://github.com/apache/commons-compress/pull/31 COMPRESS-400 It should be possible for users to create and access extra PAX headers to tar archives Add extra header map to tar archive entry. Move PAX header processing code from Tar

[GitHub] commons-text pull request #48: TEXT-88: Fixed issue with empty delimiters ar...

2017-06-12 Thread ameyjadiye
GitHub user ameyjadiye opened a pull request: https://github.com/apache/commons-text/pull/48 TEXT-88: Fixed issue with empty delimiters array with capitalizeFully You can merge this pull request into a Git repository by running: $ git pull https://github.com/ameyjadiye/commons

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread schaumb
Github user schaumb commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121406356 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,12 @@ public void testURLPattern() throws Exception

[GitHub] commons-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-12 Thread schaumb
Github user schaumb commented on a diff in the pull request: https://github.com/apache/commons-cli/pull/12#discussion_r121406356 --- Diff: src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java --- @@ -159,13 +161,12 @@ public void testURLPattern() throws Exception

Re: OSGI Version at Package Level : some more details about the COMPRESS-399 pull request

2017-06-12 Thread Simon Spero
On Mon, Jun 12, 2017 at 10:53 AM, Matt Sicker wrote: > I'd love to have a semantic versioning plugin like that which can suggest > what the version number is for the entire project. Elm (programming > language) has a tool like that, and it works rather well in practice (even > though there are so

Re: OSGI Version at Package Level : some more details about the COMPRESS-399 pull request

2017-06-12 Thread Simon Spero
On Sun, Jun 11, 2017 at 7:37 PM, Gary Gregory wrote: > Is one upshot of this is that we should base the version number based on > this OSGi report tool? > There are a few choices so the SEF upshot selector

[GitHub] commons-collections pull request #23: COLLECTIONS-606:Added build and covera...

2017-06-12 Thread vamsi-kavuri
GitHub user vamsi-kavuri opened a pull request: https://github.com/apache/commons-collections/pull/23 COLLECTIONS-606:Added build and coverage status @chtompki Please take a look at it Resolves: https://issues.apache.org/jira/browse/COLLECTIONS-606 You can merge th

[GitHub] commons-compress issue #27: COMPRESS-400

2017-06-12 Thread bodewig
Github user bodewig commented on the issue: https://github.com/apache/commons-compress/pull/27 Thanks, do you think you could re-create the PR without the commits that belong to #26 ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

Re: OSGi Version at Package Level

2017-06-12 Thread Bertrand Delacretaz
Hi Stefan, I'll do my best to answer from my heavy OSGi user's point of view - but as mentioned I know little about the specifics of Compress. On Mon, Jun 12, 2017 at 4:39 PM, Stefan Bodewig wrote: > ...Say we started versioning packages with 1.15 and all untouched APIs stay > at 1.14. We update

Re: OSGI Version at Package Level : some more details about the COMPRESS-399 pull request

2017-06-12 Thread Matt Sicker
I'd love to have a semantic versioning plugin like that which can suggest what the version number is for the entire project. Elm (programming language) has a tool like that, and it works rather well in practice (even though there are some popular libraries that are at version 5.0.0 already because

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread arunvinudss
Github user arunvinudss commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121417851 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: OSGi Version at Package Level

2017-06-12 Thread Stefan Bodewig
On 2017-06-07, Bertrand Delacretaz wrote: > On Wed, Jun 7, 2017 at 9:10 AM, Emmanuel Bourg wrote: >> ...Do I understand well that we would have to micro manage versions at the >> package level different from the version at the component level, and >> sometimes significantly different versions (li

[GitHub] commons-text issue #46: TEXT-85:Added CaseUtils class with camel case conver...

2017-06-12 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/46 Regarding the `WordUtils.capitalizeFully`, let's open that discussion up in a new Jira. I'll do that now. I think those questions are quite valid. --- If your project is set up for it, you can

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread arunvinudss
Github user arunvinudss commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121408959 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread arunvinudss
Github user arunvinudss commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121406140 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] commons-compress pull request #29: COMPRESS-406 BUILDING.md is missing licen...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-compress/pull/29 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

Re: [NUMBERS] : findbug failing in commons-number-core

2017-06-12 Thread Gilles
Hi Amey. On Sun, 11 Jun 2017 23:30:04 +0530, Amey wrote: Hi Gilles, On Thursday 08 June 2017 03:22 AM, Gilles wrote: Hello. On Wed, 7 Jun 2017 23:52:59 +0530, Amey Jadiye wrote: Hi, I was trying to run all checks on commons-number and found findbug is failing in Precision.java[544] FE_FLO

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121374008 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121376797 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121380327 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121374329 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121380270 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121378617 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121376091 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #46: TEXT-85:Added CaseUtils class with camel case...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/46#discussion_r121376021 --- Diff: src/main/java/org/apache/commons/text/CaseUtils.java --- @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

[GitHub] commons-text pull request #47: Improved test coverage for StrTokenizer

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-text/pull/47 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] commons-text issue #47: Improved test coverage for StrTokenizer

2017-06-12 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/47 I see no reason to not pull this in. Thanks for the contribution. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] commons-text pull request #45: Added IBM Jdk8 build support to travis-ci con...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-text/pull/45 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] commons-text pull request #45: Added IBM Jdk8 build support to travis-ci con...

2017-06-12 Thread chtompki
Github user chtompki commented on a diff in the pull request: https://github.com/apache/commons-text/pull/45#discussion_r121372102 --- Diff: .travis.yml --- @@ -21,8 +21,15 @@ jdk: - oraclejdk7 - oraclejdk8 +services: + - docker + +before_install

[GitHub] commons-text pull request #42: TEXT-83: Deprecate CsvTranslators from from c...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-text/pull/42 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[RESULT][VOTE] Release Apache Commons Lang 3.6 based on RC4

2017-06-12 Thread Benedikt Ritter
Hi, > Am 09.06.2017 um 11:56 schrieb Benedikt Ritter : > > Hello, > > we have fixed quite a few bugs and added some nice new features since Commons > Lang 3.5 was released, so I would like to release Commons Lang 3.6 based on > RC4. > > The only change compared to RC3 is a partial revert of L

Re: [VOTE] Release Apache Commons Lang 3.6 based on RC4

2017-06-12 Thread Benedikt Ritter
> Am 09.06.2017 um 11:56 schrieb Benedikt Ritter : > > Hello, > > we have fixed quite a few bugs and added some nice new features since Commons > Lang 3.5 was released, so I would like to release Commons Lang 3.6 based on > RC4. > > The only change compared to RC3 is a partial revert of LANG-

[GitHub] commons-text pull request #41: TEXT-67: use [lang]'s Range to replace if/els...

2017-06-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-text/pull/41 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is