[ https://issues.apache.org/jira/browse/KAFKA-1008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13742391#comment-13742391 ]
Jay Kreps commented on KAFKA-1008: ---------------------------------- I think it makes sense to do it as part of this patch since unmapping is the reason we need it and without it we will core dump the process under concurrency (so I don't think we can take the resizing change without the locking change). Sound reasonable? If we have the Os.isWindows check I think we can make BOTH the locking and the forced mmap clenaing all be inside the isWindows check. Thisshould fix things on windows and not functionally change perf at all on linux (which is good). > Unmap before resizing > --------------------- > > Key: KAFKA-1008 > URL: https://issues.apache.org/jira/browse/KAFKA-1008 > Project: Kafka > Issue Type: Bug > Components: core, log > Affects Versions: 0.8 > Environment: Windows, Linux, Mac OS > Reporter: Elizabeth Wei > Assignee: Jay Kreps > Labels: patch > Fix For: 0.8 > > Attachments: unmap-v4.patch > > Original Estimate: 1h > Remaining Estimate: 1h > > While I was studying how MappedByteBuffer works, I saw a sharing runtime > exception on Windows. I applied what I learned to generate a patch which uses > an internal open JDK API to solve this problem. > Following Jay's advice, I made a helper method called tryUnmap(). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira