uschindler commented on PR #14939:
URL: https://github.com/apache/lucene/pull/14939#issuecomment-3065566516

   Hi you are doing multiple things in this PR. Mainly I don't want the first 
file in the diff. This one is my main problem. Don't concentrate all code on 
one place and mix static and non static stuff in this interface implementation 
PanmaVectorUtilSupport. This class is only to implement this interface to be 
backed by 
   
   To make forbidden apis work: just add the API jars file to forbidden-apis 
class path (with some Gradle magic) then it would work. I can't find this 
anymore because the new Java code is too crazy to me. I would have liked to 
stay forbidden-apis using groovy code. Maybe @dweiss  can help. Basically the 
forbidden-apis call needs to get an advanced configuration with the API jars 
provided by the core-mrjar code. Maybe define a custom class path there that 
includes only the API jars.
   
   Then you can forbid those methods.


-- 
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: issues-unsubscr...@lucene.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to