janhoy commented on a change in pull request #1546:
URL: https://github.com/apache/lucene-solr/pull/1546#discussion_r439060787
##########
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:
A bit late, but I don't think this is necessary, as all callers will
send absolute paths. And if you ever get a relative path, resolving it with
`toAbsolutePath()` leads to it being relative to whatever CWD the app is
started with, while the typical resolving of relative `instanceDir` is to
resolve it relative to CoreContainer#coreRootDirectory.
----------------------------------------------------------------
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]
