rishabhdaim commented on PR #2886:
URL: https://github.com/apache/jackrabbit-oak/pull/2886#issuecomment-4386240406

   From Codex: 
   
   ```
   Strictly: yes, it can compile because CdCommand is Groovy, not Java, and it 
is not statically compiled. Groovy can compile an unresolved IterableUtils 
reference as a dynamic property/expression.
   
   Practically: no, you should not remove it. The source uses 
IterableUtils.limit(...) at CdCommand.groovy (line 42). Without the import at 
CdCommand.groovy (line 23), it would likely fail at runtime with 
MissingPropertyException: No such property: IterableUtils.
   
   I also checked the generated Java stub: it has empty method bodies, so its 
lack of IterableUtils does not prove the real Groovy source is safe without the 
import. Current oak-run compile succeeds with the import.
   ```


-- 
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]

Reply via email to