Github user kiszk commented on the issue:

    https://github.com/apache/spark/pull/19890
  
    @gatorsmile The current code works functionally correct since 
`[InternalCompilerException](https://github.com/janino-compiler/janino/blob/master/janino/src/main/java/org/codehaus/janino/InternalCompilerException.java#L35)`
 is defined as a subclass of `JaninoRuntimeException`. On the other hand, 
`JaninoRuntimeException` is deprecated now.  
    
    Do we replace `JaninoRuntimeException` in Spark code with 
`InternalCompilerException`?  
    I do not have strong preference.



---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to