[GitHub] nifi issue #1490: NIFI-3452: Add Wait processor Wait Mode property

2017-02-10 Thread basvank
Github user basvank commented on the issue:

https://github.com/apache/nifi/pull/1490
  
@ijokarumawak I have tested your patch on a single node with 3 the same 
flows and it does exactly what I expected and required as explained in [my 
mailing list 
entry](http://apache-nifi-users-list.2361937.n4.nabble.com/Problem-when-using-backpressure-to-distribute-load-over-nodes-in-a-cluster-tp863p903.html).
 I am not able to test in the cluster setting at this moment but I see no 
reason why it wouldn't work there.

Thanks for your effort!


---
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 #1037: NIFI-1893 Add processor for validating JSON

2017-01-03 Thread basvank
Github user basvank commented on the issue:

https://github.com/apache/nifi/pull/1037
  
@pvillard31 @mattyb149 thanks for the information, that didn't become clear 
from this PR.
It seems to be possible to implement in an ASF-friendly way using 
https://github.com/daveclayton/json-schema-validator (ASL 2.0), which depends 
on https://github.com/FasterXML/jackson-core (Apache 2.0) instead of json.org.
I will see if I can find the time to do this myself, but if not it can 
serve as a pointer for others.


---
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 #1037: NIFI-1893 Add processor for validating JSON

2017-01-03 Thread basvank
Github user basvank commented on the issue:

https://github.com/apache/nifi/pull/1037
  
@pvillard31 (and @bartoszjkwozniak) is it possible that something went 
wrong when merging this feature, maybe because the checks failed? It seems the 
ValidateJson processor did not end up in the master branch or any of the 
releases (I'm on 1.1.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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---