dsmiley commented on a change in pull request #1546:
URL: https://github.com/apache/lucene-solr/pull/1546#discussion_r439083930
##########
File path: solr/core/src/java/org/apache/solr/core/CoreDescriptor.java
##########
@@ -182,7 +182,7 @@ public CoreDescriptor(String coreName, CoreDescriptor
other) {
*/
public CoreDescriptor(String name, Path instanceDir, Map<String, String>
coreProps,
Properties containerProperties, ZkController
zkController) {
- this.instanceDir = instanceDir;
+ this.instanceDir = instanceDir.toAbsolutePath();
Review comment:
That makes sense; thanks. I see that all callers send absolute paths
already. So I think I should change this to throw an exception if it isn't
absolute to protect us from mistakes.
I'll push a new commit here for review. I've never done that for an already
merged PR; we'll see how it goes.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
