-
Notifications
You must be signed in to change notification settings - Fork 627
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SOLR-15787: Fix FileSystemConfigSetService test failure on Windows #1239
SOLR-15787: Fix FileSystemConfigSetService test failure on Windows #1239
Conversation
solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java
Outdated
Show resolved
Hide resolved
solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java
Outdated
Show resolved
Hide resolved
solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java
Outdated
Show resolved
Hide resolved
Tested the latest changes on Mac and Windows and both pass ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey I'm glad you stepped up to take over -- I was about to do so tonight after you got the ball rolling yesterday.
solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java
Outdated
Show resolved
Hide resolved
solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java
Outdated
Show resolved
Hide resolved
solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java
Outdated
Show resolved
Hide resolved
solr/core/src/java/org/apache/solr/core/FileSystemConfigSetService.java
Outdated
Show resolved
Hide resolved
solr/core/src/test/org/apache/solr/core/TestConfigSetService.java
Outdated
Show resolved
Hide resolved
solr/core/src/test/org/apache/solr/core/TestConfigSetService.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test passed on my windows laptop
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 Thanks @risdenk for finishing this and @NazerkeBS for starting this!
No CHANGES.txt necessary because this is really just an update to an existing entry there which was never released.
…1239) Co-authored-by: Nazerke Seidan <nseidan@salesforce.com> Co-authored-by: Kevin Risden <krisden@apache.org>
https://issues.apache.org/jira/browse/SOLR-15787