Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/20779
Sorry for the late comment. This PR itself is LGTM. I'd just like to make
some side comments on why this bug is happening.
Janino uses a peculiar encoding of implementing bridge methods for extended
accessibility from an inner class to members of its enclosing class. Here we're
actually hitting a bug in Janino where it missed creating bridge methods on the
enclosing class (`GeneratedClass...`) for `protected` members that it inherited
from a base class (`BufferedRowIterator`).
I've seen this bug in Janino before, and I plan to fix it in Janino soon.
Once it's fixed in Janino, we can safely use `protected` members such as
`append` and `stopEarly` in nested classes within `GeneratedClass...` again.
Would anybody be interested in switching these methods back to `protected` once
it's fixed in Janino and Spark bumps the Janino dependency to that new version?
Thanks!
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
