Github user barrotsteindev commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/428#discussion_r207019625
  
    --- Diff: solr/core/src/java/org/apache/solr/core/ConfigSetService.java ---
    @@ -212,12 +213,12 @@ public SchemaCaching(SolrResourceLoader loader, Path 
configSetBase) {
           super(loader, configSetBase);
         }
     
    -    public static final DateTimeFormatter cacheKeyFormatter = 
DateTimeFormat.forPattern("yyyyMMddHHmmss");
    +    public static final DateTimeFormatter cacheKeyFormatter = 
DateTimeFormatter.ofPattern("yyyyMMddHHmmss");
     
         public static String cacheName(Path schemaFile) throws IOException {
           long lastModified = Files.getLastModifiedTime(schemaFile).toMillis();
           return String.format(Locale.ROOT, "%s:%s",
    -                            schemaFile.toString(), 
cacheKeyFormatter.print(lastModified));
    +                            schemaFile.toString(), 
Instant.ofEpochMilli(lastModified).atZone(ZoneId.systemDefault()).format(cacheKeyFormatter));
    --- End diff --
    
    Sure thing,
    I'll fix it ASAP.
    Um I think this was my mistake I simply overlooked that. Shouldn't cause 
any problems, as long as the addschema test passes 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to