nifi-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [nifi] levilentz commented on pull request #4204: NIFI-7355 Added Gremlin bytecode client service.
Date Fri, 02 Oct 2020 01:15:38 GMT

levilentz commented on pull request #4204:
URL: https://github.com/apache/nifi/pull/4204#issuecomment-702476413


   > @levilentz can you take a look at this and get back to me on what improvements you
made need to be incorporated?
   
   @MikeThomsen, the main changes that I have are the following: 
   
   1. Allow for custom Jar specification 
   2. Allow for custom Class import 
   3. Allow for specification of Yaml connection string or file
   4. Logic to allow for more complex object returns from the groovy script (right now it
only plays well with simple-ish objects) 
   
   1, 2, and 3 are really important for connecting to services such as Janusgraph that are
"gremlin-compliant" but are more efficient when you use their encoders/decoders. 4 is a nice
to have and might be outside the scope of this Ticket. 
   
   Let me know what you think and if you want me to add that code into this branch. 
   
   Thanks for all your work on this! This will be a really nice addition to the graph bundle.



----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message