risdenk commented on a change in pull request #324:
URL: https://github.com/apache/solr/pull/324#discussion_r794672231
##########
File path: gradle/documentation/render-javadoc.gradle
##########
@@ -151,8 +151,25 @@ configure(project(":solr:test-framework")) {
"org.apache.solr.analysis",
"org.apache.solr.cloud",
"org.apache.solr.core",
+ "org.apache.solr.handler",
"org.apache.solr.handler.component",
"org.apache.solr.update.processor",
+ "org.apache.solr.util",
+ "org.apache.solr.search.similarities",
+ "org.apache.solr.search.function",
+ "org.apache.solr.search.facet",
+ "org.apache.solr.schema"
Review comment:
Ok so I looked into this. A handful of classes were moved from solr core
into solr test-framework due to limitations in gradle not being able to share
test classes. The move ended up in a few more split packages - which causes
issues in the render-javadoc. So these exclusions are due to the new split
packages. I didn't see a simple way to address this related to this change.
I think test-framework should probably look at how it does packages (maybe
need to at `.test.` or something to the package name) to avoid split packages.
--
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...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
