janhoy commented on a change in pull request #593:
URL: https://github.com/apache/solr/pull/593#discussion_r798965661
##########
File path: solr/core/src/java/org/apache/solr/handler/CatStream.java
##########
@@ -98,7 +98,11 @@ public void setStreamContext(StreamContext context) {
this.chroot = core.getCoreContainer().getUserFilesPath();
if (! Files.exists(chroot)) {
- throw new IllegalStateException(chroot + " directory used to load files
must exist but could not be found!");
+ try {
Review comment:
Hmm. So I reverted to the old code, simply thowing an exception and not
trying to create anything. However, that is a back-compat break. Earlier, this
exception would never be throws since the folder would always exist. Don't know
what CatStream is or whether it makes sense to use it with an empty / no
'userfiles' though.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
