levilentz commented on PR #7677: URL: https://github.com/apache/nifi/pull/7677#issuecomment-1721841706
> +1 for going to Groovy 4, I'll review Joe's PR too. Are you going to switch from gremlin-groovy to gremlin-java? What is the impact for things like [Janusgraph](https://docs.janusgraph.org/) that only appear to support Gremlin as the query language? @mattyb149 to answer this it really depends on what is going to happen to the other scripted processors that use groovy. Do you have any guidance on what that will look like for nifi as a whole? In my opinion, depending on the guidance, moving away from groovy will have minimal impact on this. For the scripted version of this processor, there is no groovy dependency. For the bytecode version, I have been running a custom build of this processor for ~ 3 years, which leverages java-gremlin to load data into janusgraph. As long as I have a way to inject the graph traversal source, it shouldent really matter if the underlying code is compiled groovy or a set of standardized methods on a custom jar. How we do that will just depend on what nifi is doing with the other groovy-injected processors such as: https://github.com/apache/nifi/blob/main/nifi-nar-bundles/nifi-groovyx-bundle/nifi-groovyx-processors/src/main/java/org/apache/nifi/processors/groovyx/ExecuteGroovyScript.java -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
