[
https://issues.apache.org/jira/browse/ZOOKEEPER-1835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13889092#comment-13889092
]
Bruno Freudensprung commented on ZOOKEEPER-1835:
------------------------------------------------
Hi Flavio,
{quote}AtomicFileWritingIdiomTest passes for me on windows, but ReconfigTest
fails. When you say: [...] Do you mean unit tests or tests you've written
yourself{quote}
I mean a test scenario that I am playing manually on Windows (and that did not
work without the patch). This manual test is described here:
https://issues.apache.org/jira/browse/ZOOKEEPER-1691?focusedCommentId=13846779&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13846779
Concerning the ReconfigTest unit test (trunk+patch), on Linux and Windows the
result is:
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0
> dynamic configuration file renaming fails on Windows
> ----------------------------------------------------
>
> Key: ZOOKEEPER-1835
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1835
> Project: ZooKeeper
> Issue Type: Bug
> Components: quorum
> Affects Versions: 3.5.0
> Environment: Windows 7 64-bit, Oracle Java 1.6.0_32-b05
> Reporter: Bruno Freudensprung
> Assignee: Bruno Freudensprung
> Fix For: 3.5.0
>
> Attachments: ZOOKEEPER-1835.patch, ZOOKEEPER-1835.patch,
> ZOOKEEPER-1835.patch, ZOOKEEPER-1835.patch
>
>
> On Windows, reconfig fails to rename the tmp dynamic config file to the real
> dynamic config filename.
> Javadoc of java.io.File.renameTo says the behavior is highly plateform
> dependent, so I guess this should not be a big surprise.
> The problem occurs in
> src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerConfig.java that
> could be modified like this:
> + curFile.delete();
> if (!tmpFile.renameTo(curFile)) {
> + configFile.delete();
> if (!tmpFile.renameTo(configFile)) {
> As suggested by Alex in https://issues.apache.org/jira/browse/ZOOKEEPER-1691
> (btw there is more information about my test scenario over there) it is a bit
> "scary" to delete the current configuration file.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
