[ https://issues.apache.org/jira/browse/CASSANDRA-13333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15937981#comment-15937981 ]
Benjamin Lerer commented on CASSANDRA-13333: -------------------------------------------- [~mkjellman] Thanks for the reviews. bq. 1. Should the loading of {{Native.register("winmm")}} in {{WindowsTimer}} also be moved into NativeLibraryWindows? {{WindowsTimer}} is really specific to Windows and according to [~JoshuaMcKenzie] [comment|https://issues.apache.org/jira/browse/CASSANDRA-13333?focusedCommentId=15929978&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15929978] we should not prevent startup due to an inability to access the {{winmm.dll}} library. So, I would be in favor of keeping it separeted for now. bq. 2. Looks like the trunk patch didn't get pushed up or potentially just a copy paste error? Currently it's just pointing at blerer/trunk. Sorry for that. It is a copy paste mistake. I fixed it. bq. 3. Thanks for putting the MSDN API URL in the method javadoc. I am pretty sure that otherwise, I will have to end up googling it in a month or two ;-) bq. 4. In NativeLibraryWindows I think the following logger statements could be simplified: I have pushed a new commit to fix it in all the branches. > Cassandra does not start on Windows due to 'JNA link failure' > ------------------------------------------------------------- > > Key: CASSANDRA-13333 > URL: https://issues.apache.org/jira/browse/CASSANDRA-13333 > Project: Cassandra > Issue Type: Bug > Reporter: Benjamin Lerer > Assignee: Benjamin Lerer > Priority: Blocker > > Cassandra 3.0 HEAD does not start on Windows. The only error in the logs is: > {{ERROR 16:30:10 JNA failing to initialize properly.}} -- This message was sent by Atlassian JIRA (v6.3.15#6346)