Issue #1176: SolrUpdateGenerator has many printStackTrace() calls.

Class members should always be prefixed with `this.` to clearly distinguish them from local variables. This way, whenever you see `this.thing` you instantly know that it's a property of the object, and just `thing` is a local variable. Writing code that breaks this assumption is bad.