[ 
https://issues.apache.org/jira/browse/SOLR-14426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17104717#comment-17104717
 ] 

Mike Drob commented on SOLR-14426:
----------------------------------

bq. Moving these classes to within another class disturbed the API (i.e. the 
very identity of these classes).

This is for 9.0, so I was under the impression that some API change is 
acceptable? Probably needs a CHANGES entry though, which I didn't include.

bq. The other solution I don't see discussed here is giving them their own 
source file

If I understand things correctly, then this wouldn't actually change the API, 
right?


I think a good choice here would be to stick with inner classes on the 
analytics contrib module, since I do not believe those classes are instantiated 
directly. And then for core classes, we can do separate source files?

> forbidden api error during precommit DateMathFunction
> -----------------------------------------------------
>
>                 Key: SOLR-14426
>                 URL: https://issues.apache.org/jira/browse/SOLR-14426
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: Build
>            Reporter: Mike Drob
>            Assignee: Mike Drob
>            Priority: Major
>             Fix For: master (9.0)
>
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> When running `./gradlew precommit` I'll occasionally see
> {code}
> * What went wrong:
> Execution failed for task ':solr:contrib:analytics:forbiddenApisMain'.
> > de.thetaphi.forbiddenapis.ForbiddenApiException: Check for forbidden API 
> > calls failed while scanning class 
> > 'org.apache.solr.analytics.function.mapping.DateMathFunction' 
> > (DateMathFunction.java): java.lang.ClassNotFoundException: 
> > org.apache.solr.analytics.function.mapping.DateMathValueFunction (while 
> > looking up details about referenced class 
> > 'org.apache.solr.analytics.function.mapping.DateMathValueFunction')
> {code}
> `./gradlew clean` fixes this, but I don't understand what or why this 
> happens. Feels like a gradle issue?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to