lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ishan Chattopadhyaya (JIRA)" <>
Subject [jira] [Commented] (SOLR-7158) ZkConfigManager filesystem fixes for Windows
Date Wed, 25 Feb 2015 11:26:04 GMT


Ishan Chattopadhyaya commented on SOLR-7158:

Ah, I see what you mean! Actually, because of the second issue with the path separator, something
like velocity\plain-hit.vm was not being considered as a subpath, and thus the Files.write()
was attempting to create a single file with that name (velocity\plain-hit.vm), which failed.
Hence, I tried to explicitly create the sub directory (velocity) by calling createDirectories(filename.getParent()),
which in view of the path separator fix seems redundant. :-) Thanks for the catch!

> ZkConfigManager filesystem fixes for Windows
> --------------------------------------------
>                 Key: SOLR-7158
>                 URL:
>             Project: Solr
>          Issue Type: Bug
>            Reporter: Ishan Chattopadhyaya
>            Assignee: Alan Woodward
>         Attachments: SOLR-7158.patch
> There were a couple of issues with ZkConfigManager:
> * Directories are not created recursively in Windows with nio.Files.write(). Need to
pre-create the directory structure while downloading config.
> * The separator for zk nodes is '/'. However, on Windows, while uploading the relative
files nested within a directory (e.g. velocity\hit-plain.vm) contain '\'. This, apart from
causing an inconsistency as compared to zk on posix systems, messed up the ZkCLITest on Windows
where count of files in zk and files in the filesystem was compared.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message