Github user pvillard31 commented on the issue:

    https://github.com/apache/nifi/pull/502
  
    @mans2singh I added some minor comments. You also need to add a NOTICE file 
for licensing aspects in the nar project (you may want to have a look to other 
processors).
    
    I just tried to run the processor on my laptop with a local running Ignite 
instance (1.6.0). It seems to behave as expected. I didn't run into issues like 
I did the last time. Regarding the comments/suggestions you made, I'm not very 
familiar with Ignite and I am not the best one to judge. I think that once all 
remarks are addressed we can get this merged into master and see if changes are 
needed depending on how this processor is used.
    
    However, regarding point 4 (logs) did you make any progress? I am not sure 
we want all the Ignite client logs into the NiFi log file.


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to