This is a sister question related to my previous post here: Simple web server when high-concurrency is met
The interview question is:
public class CounterServlet extends HttpServlet{
private volatile static int counter=0;
public int getCounter()
{
return counter;
}
public void service(HttpServletRequest request
HttpServletResponse response) throws IOException
{
counter++;
PrintWriter out=response.getWriter();
out.write("hello");
}
What issue will there be in the code above when high concurrency is encountered? My analysis is:Servlet is singleton, thus there will be issues in synchronization.It has declared the counter as volatile, which will not prevent the issue.I suggest synchronize the service method?
If you access a static values through multiple thread each thread can have it's local cached copy! To avoid this you can declare the variable as static volatile and this will force the thread to read each time the global value. However, volatile is not a substitute for proper synchronisation!
You need to synchronize the code, although you are simply doing an increment on the counter but that does not mean that the entire method will be atomic. There may be multiple threads incrementing it at the same time using the current value from the registers. It may lead to undesirable result.
You need to either synchronize the method or use the AtomicInteger for such a trivial operation.