[
https://issues.apache.org/jira/browse/NUTCH-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Enis Soztutar updated NUTCH-471:
Attachment: NutchBeanCreationSync_v2.patch
From http://www-128.ibm.com/developerworks/java/library/j-dcl.html
The bottom line is that double-checked locking, in whatever form, should not be
used because you cannot guarantee that it will work on any JVM implementation.
JSR-133 is addressing issues regarding the memory model, however,
double-checked locking will not be supported by the new memory model.
Therefore, you have two options:
* Accept the synchronization of a getInstance() method as shown in Listing
2.
* Forgo synchronization and use a static field.
We don't want to remise performance in NutchBean.get(), synchronization is not
a solution. Thus as Sami has suggested, i have written a ServetContextListener
and added NutchBean construction code there. And modified web.xml to register
the event listener class. Also In the servlet initialization, the Configuration
object is initialized and cached by NutchConfiguration, so we avoid the same
problem in NutchConfiguration.get().
i have tested the implementation and it seems OK.
Fix synchronization in NutchBean creation
-
Key: NUTCH-471
URL: https://issues.apache.org/jira/browse/NUTCH-471
Project: Nutch
Issue Type: Bug
Components: searcher
Affects Versions: 1.0.0
Reporter: Enis Soztutar
Fix For: 1.0.0
Attachments: NutchBeanCreationSync_v1.patch,
NutchBeanCreationSync_v2.patch
NutchBean is created and then cached in servlet context. But
NutchBean.get(ServletContext app, Configuration conf) is not syncronized,
which causes more than one instance of the bean (and
DistributedSearch$Client) if servlet container is accessed rapidly during
startup.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.