[GitHub] commons-text pull request #50: TEXT-90: Add CharacterPredicates for ASCII le...

2017-06-14 Thread kinow
Github user kinow commented on a diff in the pull request: https://github.com/apache/commons-text/pull/50#discussion_r122105816 --- Diff: src/test/java/org/apache/commons/text/CharacterPredicatesTest.java --- @@ -44,4 +44,77 @@ public void testDigits() {

[GitHub] commons-text pull request #50: TEXT-90: Add CharacterPredicates for ASCII le...

2017-06-14 Thread jvz
Github user jvz commented on a diff in the pull request: https://github.com/apache/commons-text/pull/50#discussion_r122104232 --- Diff: src/test/java/org/apache/commons/text/CharacterPredicatesTest.java --- @@ -44,4 +44,77 @@ public void testDigits() {

[GitHub] commons-text pull request #50: TEXT-90: Add CharacterPredicates for ASCII le...

2017-06-14 Thread jvz
Github user jvz commented on a diff in the pull request: https://github.com/apache/commons-text/pull/50#discussion_r122104071 --- Diff: src/main/java/org/apache/commons/text/CharacterPredicates.java --- @@ -48,5 +48,73 @@ public boolean test(int codePoint) { public

[GitHub] commons-text issue #51: TEXT-91: RandomStringGenerator should be able to gen...

2017-06-14 Thread jvz
Github user jvz commented on the issue: https://github.com/apache/commons-text/pull/51 Looks good to me! --- 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,

Re: Wondering VFS will support Read/Write on Google Cloud Storage?

2017-06-14 Thread 楊閔富
Another question is that VFS cannot identify directory mounted by fuse? 2017-06-15 8:47 GMT+08:00 楊閔富 : > Hi > > I am using Zeppelin on Google Cloud DataProc and want make the notebooks > to be stored on the Google Cloud Storage. > At first, I used

Wondering VFS will support Read/Write on Google Cloud Storage?

2017-06-14 Thread 楊閔富
Hi I am using Zeppelin on Google Cloud DataProc and want make the notebooks to be stored on the Google Cloud Storage. At first, I used `gcsfuse`(https://github.com/GoogleCloudPlatform/gcsfuse) to mount my Google Cloud Storage bucket as a local directory, but VFS used by Zeppelin to manage files

[GitHub] commons-text issue #45: Added IBM Jdk8 build support to travis-ci configurat...

2017-06-14 Thread chtompki
Github user chtompki commented on the issue: https://github.com/apache/commons-text/pull/45 Hm. I'll give this a look. --- 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

[GitHub] commons-text issue #45: Added IBM Jdk8 build support to travis-ci configurat...

2017-06-14 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-text/pull/45 I guess this failure is also the reason that the coveralls integration is broken. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] commons-text issue #45: Added IBM Jdk8 build support to travis-ci configurat...

2017-06-14 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-text/pull/45 This does not seem to really work. It looks like ibm jdk build is run after each (oracle8, openjdk(7|8)) build and actually fails, but the build is still marked successful, e.g.:

[GitHub] commons-text pull request #51: TEXT-91: RandomStringGenerator should be able...

2017-06-14 Thread PascalSchumacher
GitHub user PascalSchumacher opened a pull request: https://github.com/apache/commons-text/pull/51 TEXT-91: RandomStringGenerator should be able to generate a String wi… …th a random length You can merge this pull request into a Git repository by running: $ git pull

[GitHub] commons-text pull request #50: TEXT-90: Add CharacterPredicates for ASCII le...

2017-06-14 Thread PascalSchumacher
GitHub user PascalSchumacher opened a pull request: https://github.com/apache/commons-text/pull/50 TEXT-90: Add CharacterPredicates for ASCII letters (uppercase/lowerca… …se) and Arabic numerals You can merge this pull request into a Git repository by running: $ git pull

Re: Commons sub project for parallel method execution

2017-06-14 Thread Arun Mohan
Thanks for the tip Gary. Will give it a try. On Wed, Jun 14, 2017 at 12:13 PM, Gary Gregory wrote: > Briefly: If you are considering code generation, then you can do away with > using reflection. > > G > > On Wed, Jun 14, 2017 at 12:04 PM, Arun Mohan

Re: Commons sub project for parallel method execution

2017-06-14 Thread Gary Gregory
Briefly: If you are considering code generation, then you can do away with using reflection. G On Wed, Jun 14, 2017 at 12:04 PM, Arun Mohan wrote: > I was exploring ways on how to substitute the typing of method names in the > api with something thats more clean and

Re: Commons sub project for parallel method execution

2017-06-14 Thread Arun Mohan
I was checking out how the library would plug into Spring and other frameworks. I created a sample Spring project with a couple of auto wired service classes. To fetch and combine data from multiple service classes in parallel, the Spring injected service dependencies are passed to the library.

Re: Commons sub project for parallel method execution

2017-06-14 Thread Arun Mohan
I was exploring ways on how to substitute the typing of method names in the api with something thats more clean and maintainable. Using annotations, how can I provide clients the ability to specify which method needs to be specified? Any ideas? Sort of stuck on this now. Right now I am thinking

Re: [jelly] Support for multiple build tools

2017-06-14 Thread Oliver Heger
Hi, Am 14.06.2017 um 09:57 schrieb Jörg Schaible: > Hi Oliver, > > Oliver Heger wrote: > >> Hi, >> >> locally I reworked the build of Jelly core to follow the Maven standard >> layout. This of course renders all other build files dysfunctional. >> >> Are there any objections to drop these

Re: [ANNOUNCE] Apache Commons FileUpload 1.3.3 released.

2017-06-14 Thread Gary Gregory
Then maybe we need to document the fact that our code is still based on the old RFC (is it?) and that we welcome contributions to modernize to the new RFC... On Jun 14, 2017 7:07 AM, "Julian Reschke" wrote: > ... > >> The Apache Commons FileUpload library parses HTTP

Re: [ANNOUNCE] Apache Commons FileUpload 1.3.3 released.

2017-06-14 Thread Julian Reschke
... The Apache Commons FileUpload library parses HTTP requests which conform to RFC 1867, "Form-based File Upload in HTML." That is, if an HTTP request is submitted using the POST method, and with a content type of "multipart/form-data," then FileUpload can parse that request, and make the

Re: [CLI] Please review CLI-277

2017-06-14 Thread Simon Spero
But BC, per initial response. I said I needed more coffee :) On Wed, Jun 14, 2017 at 8:55 AM, Simon Spero wrote: > Oops -I missed that the method was already generic. Need bigger > tablet/more coffee / bigger caffeine tablet > > > On Jun 14, 2017 8:46 AM, "Simon Spero"

[ANNOUNCE] Apache Commons FileUpload 1.3.3 released.

2017-06-14 Thread Rob Tompkins
The Apache Commons Team is pleased to announce the release of Apache Commons FileUpload 1.3.3. The Apache Commons FileUpload library parses HTTP requests which conform to RFC 1867, "Form-based File Upload in HTML." That is, if an HTTP request is submitted using the POST method, and with a content

Re: [CLI] Please review CLI-277

2017-06-14 Thread Simon Spero
Oops -I missed that the method was already generic. Need bigger tablet/more coffee / bigger caffeine tablet On Jun 14, 2017 8:46 AM, "Simon Spero" wrote: On Jun 14, 2017 4:28 AM, "Benedikt Ritter" wrote: I’d like to have feedback for CLI-277 [1],

[CLI] Please review CLI-277

2017-06-14 Thread Simon Spero
On Jun 14, 2017 4:28 AM, "Benedikt Ritter" wrote: I’d like to have feedback for CLI-277 [1], especially whether it will affect BC. Clirr is happy with this changes, to I suppose it is fine to merge this? [1] https://github.com/apache/commons-cli/pull/13 <

[GitHub] commons-cli issue #13: CLI-277: Add generics to TypeHandler

2017-06-14 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/commons-cli/pull/13 +1 code in pull request looks good, tests too. But when I checked out the code to read it on Eclipse, I thought the class Javadoc a bit confusing. ``` * This is a temporary

[GitHub] commons-cli issue #13: CLI-277: Add generics to TypeHandler

2017-06-14 Thread kinow
Github user kinow commented on the issue: https://github.com/apache/commons-cli/pull/13 +1 code in pull request looks good, tests too. But when I checked out the code to read it on Eclipse, I thought the class Javadoc a bit confusing. ``` * This is a temporary

[GitHub] commons-cli issue #13: CLI-277: Add generics to TypeHandler

2017-06-14 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/13 [![Coverage Status](https://:/builds/11962867/badge)](https://:/builds/11962867) Coverage remained the same at 96.244% when pulling **c17d0ff553fd34e2e53e0446083ee284ee97f6cb on

[GitHub] commons-cli issue #13: CLI-277: Add generics to TypeHandler

2017-06-14 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-cli/pull/13 [![Coverage Status](https://:/builds/11962867/badge)](https://:/builds/11962867) Coverage remained the same at 96.244% when pulling **c17d0ff553fd34e2e53e0446083ee284ee97f6cb on

[CLI] Please review CLI-277

2017-06-14 Thread Benedikt Ritter
Hi, I’d like to have feedback for CLI-277 [1], especially whether it will affect BC. Clirr is happy with this changes, to I suppose it is fine to merge this? Thank you, Benedikt [1] https://github.com/apache/commons-cli/pull/13

[GitHub] commons-cli pull request #13: CLI-277: Add generics to TypeHandler

2017-06-14 Thread britter
GitHub user britter opened a pull request: https://github.com/apache/commons-cli/pull/13 CLI-277: Add generics to TypeHandler You can merge this pull request into a Git repository by running: $ git pull https://github.com/britter/commons-cli CLI-277 Alternatively you can

Re: [jelly] Support for multiple build tools

2017-06-14 Thread Jörg Schaible
Hi Oliver, Oliver Heger wrote: > Hi, > > locally I reworked the build of Jelly core to follow the Maven standard > layout. This of course renders all other build files dysfunctional. > > Are there any objections to drop these files? Maven 1 is probably no > longer in use anywhere. Most other

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

2017-06-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-cli/pull/12 --- 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-cli pull request #12: [CLI-274] implement EXISTING_FILE_VALUE type h...

2017-06-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/commons-cli/pull/12 --- 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

[ANNOUNCE] Apache Commons Lang 3.6 released!

2017-06-14 Thread Benedikt Ritter
The Apache Commons community is happy to announce the availability of Apache Commons Lang 3.6. Apache Commons Lang provides helper utilities for the java.lang API, notably String manipulation methods, basic numerical methods, object reflection, concurrency, creation and serialization and

[GitHub] commons-compress issue #33: add-some-Unit-Tests Added some Unit Tests to inc...

2017-06-14 Thread bodewig
Github user bodewig commented on the issue: https://github.com/apache/commons-compress/pull/33 Fortunately I'm not the only one who could merge this :-) More seriously, it may take a bit of time until I get there, but I will. --- If your project is set up for it, you can

[GitHub] commons-text issue #49: TEXT-89: UTF-32 support for WordUtils.initials using...

2017-06-14 Thread arunvinudss
Github user arunvinudss commented on the issue: https://github.com/apache/commons-text/pull/49 @ecki I agree and thats the best we can do for now to support UTF-32 . Thanks for the input 👍 --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] commons-text issue #49: TEXT-89: UTF-32 support for WordUtils.initials

2017-06-14 Thread ecki
Github user ecki commented on the issue: https://github.com/apache/commons-text/pull/49 I think using UTF-32 to describe UTF-16 with surrogate pairs is a bit misleading. So I would adjust subject and comment, as this does not enable fixed-4-byte characters. --- If your project is

[GitHub] commons-text pull request #49: TEXT-89: UTF-32 support for WordUtils.initial...

2017-06-14 Thread arunvinudss
GitHub user arunvinudss opened a pull request: https://github.com/apache/commons-text/pull/49 TEXT-89: UTF-32 support for WordUtils.initials @chtompki Adding support for surrogate pairs to WordUtils.initials. Characters outside BMP can be used now and also added unit tests for