[GitHub] nifi issue #2988: NIFI-5574 Removed usage of Paths.get() due to TEST_RES_PAT...

2018-09-19 Thread nalewis
Github user nalewis commented on the issue: https://github.com/apache/nifi/pull/2988 Yep all good there! ``` [INFO] --- [INFO] T E S T S [INFO] --- [INFO

[GitHub] nifi issue #2988: NIFI-5574 Removed usage of Paths.get() due to TEST_RES_PAT...

2018-09-18 Thread nalewis
Github user nalewis commented on the issue: https://github.com/apache/nifi/pull/2988 @jtstorck Got dragged away to other projects and wanted to get back to this. Trying to build on your branch at the moment. With `mvn clean install -T C2 -DskipTests` I can successfully build, however

[GitHub] nifi issue #2988: NIFI-5574 Removed usage of Paths.get() due to TEST_RES_PAT...

2018-09-05 Thread nalewis
Github user nalewis commented on the issue: https://github.com/apache/nifi/pull/2988 I'm on Windows 10. ---

[GitHub] nifi issue #2988: NIFI-5574 Removed usage of Paths.get() due to TEST_RES_PAT...

2018-09-05 Thread nalewis
Github user nalewis commented on the issue: https://github.com/apache/nifi/pull/2988 I grabbed the branch and attempted a clean install this morning. It failed with the following: mvn --version ``` Apache Maven 3.5.2 (138edd61fd100ec658bfa2d307c43b76940a5d7d

[GitHub] nifi issue #2987: NIFI-3344 Added property to JoltTransformJSON allowing the...

2018-09-04 Thread nalewis
Github user nalewis commented on the issue: https://github.com/apache/nifi/pull/2987 Went ahead and made the issue based on our discussion. Thanks @alopresto . https://issues.apache.org/jira/browse/NIFI-5574 ---

[GitHub] nifi pull request #2987: NIFI-3344 Added property to JoltTransformJSON allow...

2018-09-04 Thread nalewis
Github user nalewis commented on a diff in the pull request: https://github.com/apache/nifi/pull/2987#discussion_r215029119 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/JoltTransformJSON.java

[GitHub] nifi issue #2987: NIFI-3344 Added property to JoltTransformJSON allowing the...

2018-09-04 Thread nalewis
Github user nalewis commented on the issue: https://github.com/apache/nifi/pull/2987 At the moment my windows installs fail on this NifiGroovyTest, where it tries to generate and reference a linux style path. I saw a recent email discussion that seemed to bring up a similar issue

[GitHub] nifi issue #2987: NIFI-3344 Added property to JoltTransformJSON allowing the...

2018-09-04 Thread nalewis
Github user nalewis commented on the issue: https://github.com/apache/nifi/pull/2987 I have yet to be able to successfully build the Nifi master branch without skipping tests on either my windows and CentOS machines. I built with -DskipTests=true and validated the processor there. ---

[GitHub] nifi pull request #2987: NIFI-3344 Added property to JoltTransformJSON allow...

2018-09-04 Thread nalewis
GitHub user nalewis opened a pull request: https://github.com/apache/nifi/pull/2987 NIFI-3344 Added property to JoltTransformJSON allowing the user to sp… …ecify pretty print, defaults to false Thank you for submitting a contribution to Apache NiFi. In order

[GitHub] nifi pull request #2978: NIFI-5565 Added reference to confluence documentati...

2018-08-30 Thread nalewis
GitHub user nalewis opened a pull request: https://github.com/apache/nifi/pull/2978 NIFI-5565 Added reference to confluence documentation from the developers guide Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution