[sql] Fix uninit vars in Connection.

[Closed because original issue was reverted, and wrong owner for some reason.]

[sql] Fix uninit vars in Connection.

BUG=489784
TBR=michaeln@chromium.org

[Scott Hess - ex-Googler, 2015-09-23 22:04:40]

The CQ bit was checked by shess@chromium.org

[Scott Hess - ex-Googler, 2015-09-23 22:05:39]

shess@chromium.org changed reviewers:
+ michaeln@chromium.org

[Scott Hess - ex-Googler, 2015-09-23 22:05:39]

Sometimes I hate this language.  Added instance vars, forgot to update
constructor because I was looking for kaiju not gremlins.

[michaeln, 2015-09-23 22:17:22]

https://codereview.chromium.org/1368493002/diff/1/sql/connection.cc
File sql/connection.cc (right):

https://codereview.chromium.org/1368493002/diff/1/sql/connection.cc#newcode287
sql/connection.cc:287: total_changes_at_last_release_(0),
lgtm

maybe use c++ 11 feature that lets you assign default values in the header?

[Scott Hess - ex-Googler, 2015-09-23 22:27:08]

On 2015/09/23 22:17:22, michaeln wrote:
> https://codereview.chromium.org/1368493002/diff/1/sql/connection.cc
> File sql/connection.cc (right):
> 
> https://codereview.chromium.org/1368493002/diff/1/sql/connection.cc#newcode287
> sql/connection.cc:287: total_changes_at_last_release_(0),
> lgtm
> 
> maybe use c++ 11 feature that lets you assign default values in the header?

Do we do that?  If so, maybe I'll circle back with a more comprehensive version,
as there is only one variable initialized here which actually needs to be here.

Imagine a world where you could just say "Zero fill this guy" and only
initialize the outliers.  "Don't pay for what you don't use" is fine, unless you
end up paying anyhow and you're required to use pennies.

[Scott Hess - ex-Googler, 2015-09-23 22:40:11]

The CQ bit was unchecked by shess@chromium.org