<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style>
</head>
<body class='hmmessage'>
<BR>
As a user I'm not tied to any specific implementation (I just want the feature), but I 'll have to look at the code to fully understand your suggestion. I don't have the code in front of me at the moment (I'm at work). However, I don't believe I lock anything specifically. It's very possible that I am doing something that is causing lock contention, can you point me to where this is occurring? I am accessing the same per segment lock free queue that the article mentions.<BR>
<BR>
-- Dave Marion<BR> <BR>
<HR id=stopSpelling>
Date: Mon, 31 Jan 2011 11:57:53 -0300<BR>From: vblagoje@redhat.com<BR>To: infinispan-dev@lists.jboss.org<BR>Subject: [infinispan-dev] ISPN-863 proposal<BR><BR>
<META content="Microsoft SafeHTML" name=Generator>
<P class=ecxMsoPlainText><FONT face="Courier New">Dave,<BR><BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New">I applaud you for this effort and your implementation and wiki are indeed admirable as you have followed the goals outlined in this JIRA. However, now that I have had an opportunity to look at the final solution I have one concern. </FONT><FONT face="Courier New">We went through a great lengths to design a container that has a low lock contention [1] while maintaining high precision of a sophisticated eviction algorithm. Goals outlined in this JIRA have lead you in a direction </FONT><FONT face="Courier New">to redesign the original proposal. I was wondering if it is possible to use your MemoryMonitor to make a decision whether or not to grow BCHM rather than to check if eviction should be done upon each put invocation? Currently we preset the size of BCHM and we never grow it (i.e. Segment#rehash is never invoked). Maybe this is a place where </FONT><FONT face="Courier New">MemoryMonitor comes as a natural fit. We can keep the current design as outlined in [1] while reaping the benefits of a memory bound resizeable container. <BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New"><BR>WDYT?<BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New"><BR>Regards,<BR>Vladimir<BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New"><BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New"><BR>[1] <A class=ecxmoz-txt-link-freetext href="http://infinispan.blogspot.com/2010/03/infinispan-eviction-batching-updates.html" target=_blank>http://infinispan.blogspot.com/2010/03/infinispan-eviction-batching-updates.html</A><BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New"><BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New">Dave said:<BR></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New">It took me some time to figure out how to use Git and add some more tests. I have pushed my changes to a topic branch located at <A href="https://github.com/dlmarion/infinispan/tree/ISPN-863-master" target=_blank>https://github.com/dlmarion/infinispan/tree/ISPN-863-master</A></FONT><FONT face="Courier New">. </FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New">Please let me know if you have any questions or concerns. I have put up some documentation at <A href="https://github.com/dlmarion/infinispan/wiki/ISPN-863-Implementation" target=_blank>https://github.com/dlmarion/infinispan/wiki/ISPN-863-Implementation</A></FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New"> </FONT></P>
<P class=ecxMsoPlainText><FONT face="Courier New"> </FONT></P><FONT face="Courier New">-- Dave Marion</FONT> <BR>_______________________________________________ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev </body>
</html>