[GitHub] nifi pull request #609: NIFI-2142 Cache compiled XSLT in TransformXml

2016-07-06 Thread jfrazee
GitHub user jfrazee opened a pull request:

https://github.com/apache/nifi/pull/609

NIFI-2142 Cache compiled XSLT in TransformXml

Two other tidbits:

1. This also modifies the XSLT_FILENAME attribute to support expression 
language.
2. This updated the test files to have 3 space indents because Saxon 
defaults to that and AFAICT the free version doesn't let you change that -- 
it's either no indent or 3 space. Could be wrong though.

See https://github.com/apache/nifi/pull/591 for PR against 0.x and past 
discussion.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jfrazee/nifi NIFI-2142

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/609.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #609


commit f595df477e971fd09c3a734f18a882af0d72818c
Author: Joey Frazee <joey.fra...@icloud.com>
Date:   2016-06-28T22:29:43Z

Added Guava cache for XSLT stylesheets in TransformXml




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #591: NIFI-2142 Cache compiled XSLT in TransformXml

2016-07-06 Thread jfrazee
Github user jfrazee commented on the issue:

https://github.com/apache/nifi/pull/591
  
Closing this and resubmitting against master since 0.7 is frozen.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #591: NIFI-2142 Cache compiled XSLT in TransformXml

2016-07-06 Thread jfrazee
Github user jfrazee closed the pull request at:

https://github.com/apache/nifi/pull/591


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #565: NIFI-2090 Add support for HL7 segment names and comp...

2016-06-22 Thread jfrazee
GitHub user jfrazee opened a pull request:

https://github.com/apache/nifi/pull/565

NIFI-2090 Add support for HL7 segment names and components in 
ExtractHL7Attributes



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jfrazee/nifi NIFI-2090

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/565.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #565


commit b6a2238a6d19d05e01b4ef986aee8b6e8a58746c
Author: Joey Frazee <joey.fra...@icloud.com>
Date:   2016-06-23T04:44:40Z

Added options for segment names, parse fields in ExtractHL7Attributes




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #504: NIFI-1979 Moved HL7 test file into TestExtractHL7Att...

2016-06-07 Thread jfrazee
Github user jfrazee closed the pull request at:

https://github.com/apache/nifi/pull/504


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #495: NIFI-1968 ExtractHL7Attributes is squashing empty co...

2016-06-07 Thread jfrazee
Github user jfrazee closed the pull request at:

https://github.com/apache/nifi/pull/495


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #504: NIFI-1979 Moved HL7 test file into TestExtractHL7Att...

2016-06-07 Thread jfrazee
GitHub user jfrazee opened a pull request:

https://github.com/apache/nifi/pull/504

NIFI-1979 Moved HL7 test file into TestExtractHL7Attributes to fix CRLF bug



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jfrazee/nifi NIFI-1979

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/504.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #504


commit 5156b56cff71da79fe6170d3ba888a4150fb8315
Author: Joey Frazee <joey.fra...@icloud.com>
Date:   2016-06-07T15:01:38Z

Moved HL7 test file into TestExtractHL7Attributes, fixes NIFI-1979




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #495: NIFI-1968 ExtractHL7Attributes is squashing empty co...

2016-06-04 Thread jfrazee
GitHub user jfrazee opened a pull request:

https://github.com/apache/nifi/pull/495

NIFI-1968 ExtractHL7Attributes is squashing empty components

This updates ExtractHL7Attributes and corresponding tests so it works with 
empty components (^^) in sub-segments. It uses the built in encoding 
functionality from the HAPI HL7 API so the NiFi getValue() is removed. It's 
worth noting that HL7's encoding of some fields (MSH.1 and MSH.2) aren't the 
plain text so these are unescaped in this fix. Escaping them would be a 
breaking change that arguably doesn't relate to any bug. Not sure what the 
right thing there is.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jfrazee/nifi NIFI-1968

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/495.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #495


commit 91e7584d0cb97e876d7078e50bd56d5289c12331
Author: Joey Frazee <joey.fra...@icloud.com>
Date:   2016-06-04T19:51:18Z

Fixes NIFI-1968 so ExtractHL7Attributes will keep empty components




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request: NIFI-1710 Resolve path name to nifi.sh in start...

2016-03-31 Thread jfrazee
Github user jfrazee commented on the pull request:

https://github.com/apache/nifi/pull/315#issuecomment-204017042
  
@arpitgupta Ok, yeah, older versions of coreutils don't have it. I was 
doing this with `readlink` but BSD readlink doesn't have the same options as 
GNU. Looks like it might be necessary to push this down into detectOS(), if 
people want this sort of change at all.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request: NIFI-1710 Resolve path name to nifi.sh in start...

2016-03-31 Thread jfrazee
GitHub user jfrazee opened a pull request:

https://github.com/apache/nifi/pull/315

NIFI-1710 Resolve path name to nifi.sh in start script

This adds `realpath` to nifi.sh to resolve location of nifi.sh when its 
symlinked (instead of installed using the `install` command). Recent versions 
of Linux and BSD make `realpath` available.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jfrazee/nifi NIFI-1710

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/315.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #315


commit dbad7f41d6177f38f3736e539e77bb05433c2773
Author: Joey Frazee <joey.fra...@icloud.com>
Date:   2016-03-31T15:33:29Z

Added realpath to nifi.sh to resolve its location when symlinked




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request: NIFI-1617 Add source filename metadata to Ident...

2016-03-19 Thread jfrazee
Github user jfrazee closed the pull request at:

https://github.com/apache/nifi/pull/273


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request: Added source filename metadata to IdentifyMimeT...

2016-03-12 Thread jfrazee
GitHub user jfrazee opened a pull request:

https://github.com/apache/nifi/pull/273

Added source filename metadata to IdentifyMimeType, fixes NIFI-1617

Tika mime type Detector determines  mime type on the basis of filename 
extensions in some cases (e.g., CSV). This adds the FlowFile filename attribute 
to the Detector Metadata if it's available.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jfrazee/nifi NIFI-1617

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/273.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #273


commit 2ece5d7943f63258bd3dd9abc62aa734392ecce0
Author: Joey Frazee <joey.fra...@icloud.com>
Date:   2016-03-10T22:49:59Z

Added source filename metadata to IdentifyMimeType, fixes NIFI-1617




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---