[GitHub] nifi pull request #506: NIFI-1660 - Enhance the expression language with jso...

2016-06-16 Thread ckmcd
Github user ckmcd closed the pull request at:

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


---
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 #303: NIFI-1660 - Enhance the expression language with jso...

2016-06-16 Thread ckmcd
Github user ckmcd closed the pull request at:

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


---
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 #322: NIFI-1661 add random() function to expression langua...

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

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


---
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 #303: NIFI-1660 - Enhance the expression language with jsonPath f...

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

https://github.com/apache/nifi/pull/303
  
This pull request should be good to go as all feedback has been 
incorporated.


---
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-1660 - Enhance the expression language wit...

2016-05-25 Thread ckmcd
Github user ckmcd commented on the pull request:

https://github.com/apache/nifi/pull/303#issuecomment-221658273
  
@markap14 wrote
> In the vast majority of use cases, I would expect that the JSON Path will 
be a literal string that does not reference any attributes. In this case, I 
would want to avoid compiling the JSON Path every time. Would recommend that if 
the jsonPathExp is an instance of the StringLiteralEvaluator, we pre-compile 
the JSON Path. We follow a similar pattern with the MatchesEvaluator for 
pre-compiling regular expressions, and it can make a pretty huge difference on 
performance.

Mark, can you show me an example of how to write an expression that would 
not be using a StringLiteralEvaluator?

Thanks!


---
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-1660 - Enhance the expression language wit...

2016-05-23 Thread ckmcd
Github user ckmcd commented on the pull request:

https://github.com/apache/nifi/pull/303#issuecomment-221073806
  
I've rebase on master.  Other suggested changes are in the works.


---
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-1660 - Enhance the expression language wit...

2016-05-23 Thread ckmcd
Github user ckmcd commented on the pull request:

https://github.com/apache/nifi/pull/303#issuecomment-221019823
  
@mattyb149, @markap14  I am working on your suggestions.  Thanks.


---
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-1661 add random() function to expression l...

2016-04-18 Thread ckmcd
Github user ckmcd commented on the pull request:

https://github.com/apache/nifi/pull/322#issuecomment-211429563
  
I've update the PR to address the issues (1 & 2) from above.  

1. Updated the EL Guide to use plus() instead of add()
2. Use Math.abs() so as only to generate non-negative numbers.  Updated EL 
guide to reflect this change.


---
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-1660 - Enhance the expression language wit...

2016-04-04 Thread ckmcd
Github user ckmcd commented on the pull request:

https://github.com/apache/nifi/pull/303#issuecomment-205345851
  
Please ignore the request to kick the Travis build.  I rebased and that did 
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 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-1660 - Enhance the expression language wit...

2016-03-24 Thread ckmcd
GitHub user ckmcd opened a pull request:

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

NIFI-1660 - Enhance the expression language with jsonPath function

Allows JSON path application in EL.

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

$ git pull https://github.com/ckmcd/nifi NIFI-1660

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

https://github.com/apache/nifi/pull/303.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 #303


commit 4c54a8ef9763c2cd50b44001013df27ee5778aa7
Author: Chris McDermott <chris.mcderm...@hpe.com>
Date:   2016-03-24T18:10:19Z

NIFI-1660 - Enhance the expression language with jsonPath function




---
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.
---