[GitHub] nifi issue #1906: NIFI-4046: If we are unable to parse out any records from ...

2017-06-11 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/1906 I tested with following schema: ```json { "type": "record", "name": "test", "fields": [ { "name": "id", "type": "long" }, { "name": "name", "type": [

[GitHub] nifi issue #1906: NIFI-4046: If we are unable to parse out any records from ...

2017-06-11 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/1906 Reviewing... --- 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

[jira] [Updated] (NIFI-4041) ListFTP/ListSFTP doesn't provide file.size attribute

2017-06-11 Thread Koji Kawamura (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4041?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Koji Kawamura updated NIFI-4041: Resolution: Fixed Fix Version/s: 1.4.0 Status: Resolved (was: Patch Available) >

[GitHub] nifi issue #1899: NIFI-4041: Add file.size attribute to ListFTP and ListSFTP

2017-06-11 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/1899 Thank you for reviewing @joewitt , I updated WritesAttributes annotation and merged this to master. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1899: NIFI-4041: Add file.size attribute to ListFTP and ListSFTP

2017-06-11 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/1899 cool +1 --- 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

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread cricket007
Github user cricket007 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121297796 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread taftster
Github user taftster commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121296995 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[jira] [Updated] (NIFI-3983) Support ability to make JMS 2.0 durable subscriptions on Topic

2017-06-11 Thread Koji Kawamura (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-3983?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Koji Kawamura updated NIFI-3983: Status: Patch Available (was: Open) > Support ability to make JMS 2.0 durable subscriptions on

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046162#comment-16046162 ] ASF GitHub Bot commented on NIFI-4057: -- Github user cricket007 commented on a diff in the pull

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread cricket007
Github user cricket007 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121296054 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046136#comment-16046136 ] ASF GitHub Bot commented on NIFI-4057: -- Github user taftster commented on a diff in the pull request:

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread taftster
Github user taftster commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121292352 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046135#comment-16046135 ] ASF GitHub Bot commented on NIFI-4057: -- Github user taftster commented on a diff in the pull request:

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread taftster
Github user taftster commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121292333 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046134#comment-16046134 ] ASF GitHub Bot commented on NIFI-4057: -- Github user cricket007 commented on a diff in the pull

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread cricket007
Github user cricket007 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121292308 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046131#comment-16046131 ] ASF GitHub Bot commented on NIFI-4057: -- Github user cricket007 commented on a diff in the pull

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread cricket007
Github user cricket007 commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121292271 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046122#comment-16046122 ] ASF GitHub Bot commented on NIFI-4057: -- Github user taftster commented on the issue:

[GitHub] nifi issue #1910: NIFI-4057 Docker Image is twice as large as necessary

2017-06-11 Thread taftster
Github user taftster commented on the issue: https://github.com/apache/nifi/pull/1910 Changes to this Dockerfile should also be reciprocally to ../dockermaven/Dockerfile --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046119#comment-16046119 ] ASF GitHub Bot commented on NIFI-4057: -- Github user taftster commented on a diff in the pull request:

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046121#comment-16046121 ] ASF GitHub Bot commented on NIFI-4057: -- Github user taftster commented on a diff in the pull request:

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046120#comment-16046120 ] ASF GitHub Bot commented on NIFI-4057: -- Github user taftster commented on a diff in the pull request:

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread taftster
Github user taftster commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121290876 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread taftster
Github user taftster commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121291104 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread taftster
Github user taftster commented on a diff in the pull request: https://github.com/apache/nifi/pull/1910#discussion_r121291213 --- Diff: nifi-docker/dockerhub/Dockerfile --- @@ -16,37 +16,37 @@ # under the License. # -FROM openjdk:8 -MAINTAINER Apache NiFi

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046074#comment-16046074 ] ASF GitHub Bot commented on NIFI-4057: -- GitHub user cricket007 opened a pull request:

[GitHub] nifi pull request #1910: NIFI-4057 Docker Image is twice as large as necessa...

2017-06-11 Thread cricket007
GitHub user cricket007 opened a pull request: https://github.com/apache/nifi/pull/1910 NIFI-4057 Docker Image is twice as large as necessary Resulting Docker image in now 1.29 GB instead of 2.58 GB **Detailed changes** * Merging unnecessary layers *

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread Jordan Moore (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046062#comment-16046062 ] Jordan Moore commented on NIFI-4057: Cool, yeah the page loads with the JRE, so I assumed it would be

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread Adam Taft (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046054#comment-16046054 ] Adam Taft commented on NIFI-4057: - OK, indeed. JSP compilation is being handled by the Eclipse compiler.

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread Adam Taft (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046050#comment-16046050 ] Adam Taft commented on NIFI-4057: - Yes, you're right. Using the JRE only version might be a good idea as

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread Jordan Moore (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046045#comment-16046045 ] Jordan Moore commented on NIFI-4057: Just want to mention, I'm already working on this, and have a

[jira] [Commented] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread Adam Taft (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046044#comment-16046044 ] Adam Taft commented on NIFI-4057: - This is a good catch and will be super easy to fix. There are some

[jira] [Updated] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread Jordan Moore (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4057?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jordan Moore updated NIFI-4057: --- Description: By calling {{chown}} as a secondary {{RUN}} command, you effectively double the size of

[jira] [Created] (NIFI-4057) Docker Image is twice as large as necessary

2017-06-11 Thread Jordan Moore (JIRA)
Jordan Moore created NIFI-4057: -- Summary: Docker Image is twice as large as necessary Key: NIFI-4057 URL: https://issues.apache.org/jira/browse/NIFI-4057 Project: Apache NiFi Issue Type: Bug

[GitHub] nifi pull request #1874: NiFi 3973 - Create a new Kudu Processor to ingest d...

2017-06-11 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/1874#discussion_r121285049 --- Diff: nifi-nar-bundles/nifi-kudu-bundle/nifi-kudu-processors/src/test/java/org/apache/nifi/processors/kudu/TestPutKudu.java --- @@ -0,0 +1,132 @@

[GitHub] nifi pull request #1874: NiFi 3973 - Create a new Kudu Processor to ingest d...

2017-06-11 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/1874#discussion_r121285021 --- Diff: nifi-nar-bundles/nifi-kudu-bundle/nifi-kudu-processors/src/main/java/org/apache/nifi/processors/kudu/PutKudu.java --- @@ -0,0 +1,130 @@ +/*

[GitHub] nifi pull request #1874: NiFi 3973 - Create a new Kudu Processor to ingest d...

2017-06-11 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/1874#discussion_r121284983 --- Diff: nifi-nar-bundles/nifi-kudu-bundle/nifi-kudu-processors/src/main/java/org/apache/nifi/processors/kudu/PutKudu.java --- @@ -0,0 +1,130 @@ +/*

[GitHub] nifi pull request #1874: NiFi 3973 - Create a new Kudu Processor to ingest d...

2017-06-11 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/1874#discussion_r121284976 --- Diff: nifi-nar-bundles/nifi-kudu-bundle/nifi-kudu-processors/src/main/java/org/apache/nifi/processors/kudu/PutKudu.java --- @@ -0,0 +1,130 @@ +/*

[GitHub] nifi pull request #1874: NiFi 3973 - Create a new Kudu Processor to ingest d...

2017-06-11 Thread joewitt
Github user joewitt commented on a diff in the pull request: https://github.com/apache/nifi/pull/1874#discussion_r121284912 --- Diff: nifi-nar-bundles/nifi-kudu-bundle/nifi-kudu-processors/src/main/java/org/apache/nifi/processors/kudu/AbstractKudu.java --- @@ -0,0 +1,140 @@

[jira] [Commented] (NIFI-4041) ListFTP/ListSFTP doesn't provide file.size attribute

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4041?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16046012#comment-16046012 ] ASF GitHub Bot commented on NIFI-4041: -- Github user joewitt commented on the issue:

[GitHub] nifi issue #1899: NIFI-4041: Add file.size attribute to ListFTP and ListSFTP

2017-06-11 Thread joewitt
Github user joewitt commented on the issue: https://github.com/apache/nifi/pull/1899 @ijokarumawak can you go ahead and also update the WritesAttributes annotation to cover this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Commented] (NIFI-3260) Official Docker Image

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-3260?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16045972#comment-16045972 ] ASF GitHub Bot commented on NIFI-3260: -- Github user cricket007 commented on the issue:

[GitHub] nifi issue #1372: NIFI-3260 Official Docker Image

2017-06-11 Thread cricket007
Github user cricket007 commented on the issue: https://github.com/apache/nifi/pull/1372 That also seems like a good idea. mkdir on NIFI_HOME, then chown, curl, and extract. I hit the download limit on Apache site last night (5 GB) testing some options, so I'll continue

[jira] [Commented] (NIFI-3260) Official Docker Image

2017-06-11 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-3260?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16045859#comment-16045859 ] ASF GitHub Bot commented on NIFI-3260: -- Github user taftster commented on the issue:

[GitHub] nifi issue #1372: NIFI-3260 Official Docker Image

2017-06-11 Thread taftster
Github user taftster commented on the issue: https://github.com/apache/nifi/pull/1372 One solution is just to move the chown and USER components _above_ the curl statement. Basically you want curl to be executed as the NIFI user, not as root. Just a quick test, rearranging