Github user joewitt commented on the issue:

    https://github.com/apache/nifi/pull/2671
  
    Ok i've attached a patch which helps with some aspects of POM construction, 
flagging things like resource utilization since it appears to be loading full 
content into memory, and renaming the service to indicate it is a MarkLogic 
service rather than just a database service.  There is an outstanding need to 
sort out the security configuration.  For SSLContext stuff those things should 
utilize the standard mechanism of obtaining that as you can follow from a 
number of other processors.  Also, there is a kerberos context for security 
setting but there does not appear to be any associated settings for the user.  
The security configurations should be removed in favor of simple/digest for now 
OR completed and with some consistency to other items.  For security relevant 
things CVEs become a concern so we take these more seriously.  For things about 
the performance/logic of the processor interaction with MarkLogic that we can 
improve over time if needed but security we want to get right 
 up front.  The other thing that needs to happen is the nar bundles need their 
LICENSE/NOTICE file(s) added if necessary.  I looked at one of the nars and 
there would definitely need to be entries.  Please try adding these in like 
other nars and I'm happy to help tweak it to get it to the finish line.
    
    If you have questions on how to achieve any of the above please ask.  Show 
an example nar you looked at which is similar so that we can best help close 
remaining gaps but from a place of good examples that you've looked at.
    
    Thanks


---

Reply via email to