Discard example use AtomicLong for no reason

[normanmaurer]

See stackoverflow:

http://stackoverflow.com/questions/8317766/about-handlers-in-netty-and-concurrency

[ptnptn]

I think the fix has data race because access to transferredBytes is not synchronized.

[normanmaurer]

Well it use new instance of the handler per Channel so you don't need synchronized. Or do you mean because you "can" access the written amount via a public get method which could return a not 100 % "accurate" version of the date here ?

[ptnptn]

Yes, this is what i mean since you added the getter