RequestHandlerBase statistics atomicity problem?

[Sean Timm]

public abstract class RequestHandlerBase implements 
SolrRequestHandler, SolrInfoMBean {

  // statistics
  // TODO: should we bother synchronizing these, or is an off-by-one error
  // acceptable every million requests or so?
  long numRequests;
  long numErrors;
Shouldn't numRequests and numErrors be volatile variables?  Writes are 
not atomic on longs unless they are volatile.

-Sean