Github user joetrite closed the pull request at:
https://github.com/apache/nifi/pull/2875
---
GitHub user joetrite opened a pull request:
https://github.com/apache/nifi/pull/2875
NIFI-5351 Support for Decimals in FetchParquet
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure
Github user joetrite commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2711#discussion_r193808025
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AttributesToCSV.java
---
@@ -0,0
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/2711
@bdesert good catch, we should add the option to push the header to
attribute or content based on the selection. The header should also be csv
format so that when the header and data
Github user joetrite commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2711#discussion_r190733381
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AttributesToCSV.java
---
@@ -0,0
Github user joetrite commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2711#discussion_r190733355
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AttributesToCSV.java
---
@@ -0,0
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/2711
@bdesert it looks like the regex support was added to AttributesToJSON
after copied the code. I'll add regex support to this processor and check if
anything else has changed.
---
Github user joetrite commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2711#discussion_r190288780
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AttributesToCSV.java
---
@@ -0,0
Github user joetrite commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2711#discussion_r190286828
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/AttributesToCSV.java
---
@@ -0,0
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/1589
i drove this one off a cliff :). New pull request raised.
---
GitHub user joetrite opened a pull request:
https://github.com/apache/nifi/pull/2711
NIFI-1705 - Adding AttributesToCSV processor
New pull request replacing
https://github.com/apache/nifi/pull/1589
You can merge this pull request into a Git repository by running:
$ git
Github user joetrite closed the pull request at:
https://github.com/apache/nifi/pull/1589
---
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/1589
@mattyb149 thanks for reviewing, let me have a think on these things. i
like the idea of allowing the attribute list to be dynamic.
---
If your project is set up for it, you can reply
GitHub user joetrite opened a pull request:
https://github.com/apache/nifi/pull/1589
NIFI-1705 Adding AttributesToCSV processor
NIFI-1705 Adding AttributesToCSV processor
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/joetrite
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/1564
@joewitt fixed routing, simplified code + thread safety issue and spelling
of tomato :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/1564
@joewitt i'm made most of these changes, still testings things but one
thing i want to note/raise (maybe by design?) in the original code is that if
the ff does not contain any attributes that match
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/1564
@joewitt thx for the feedback. will work on it.
---
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
Github user joetrite commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1564#discussion_r10431
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ScanAttribute.java
---
@@ -60,13
Github user joetrite closed the pull request at:
https://github.com/apache/nifi/pull/1536
---
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
Github user joetrite commented on the issue:
https://github.com/apache/nifi/pull/1536
created new pull request #1564 with single commit. closing this. thanks
@joewitt @alopresto for all the help and advise.
---
If your project is set up for it, you can reply to this email and have
GitHub user joetrite opened a pull request:
https://github.com/apache/nifi/pull/1564
NIFI-3497 ScanAttribute Tagging
NIFI-3497 Adding tagging option to ScanAttribute
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/joetrite/nifi
Github user joetrite commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1536#discussion_r104274990
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ScanAttribute.java
---
@@ -97,13
GitHub user joetrite opened a pull request:
https://github.com/apache/nifi/pull/1543
NIFI-1750 - updated dict file desc
Clarifying the load/reload strategy of the dictionary file.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user joetrite opened a pull request:
https://github.com/apache/nifi/pull/1536
NIFI-3497 ScanAttribute Tagging
Adding tagging to ScanAttribute.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/joetrite/nifi master
24 matches
Mail list logo