[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: calcite-avatica git commit: [CALCITE-2412] Add appveyor.yml to have tests on Windows against jdk1.8, jdk9, jdk10 Add Appveyor badge Add -DskipDockerCheck because of CALCITE-2385 and to make it sync with travis.yml

>I don’t think it’s possible to automate.

Well, a rule of "first line should be separated by a blank line" seems to
be automatable.
The rule of "CALCITE-XXX should be in [...]" seems to be automatable.
And so on.

>I was a bit surprised that you, as a committer, had not already committed

setDynamicParam did not look good enough to me

I'm inclined to incline Avatica to expose
TypedValue.setToPreparedStatement(PreparedStatement ps, int index) kind of
API, so ResultSetEnumerable.setDynamicParam could be removed altogether.

PS. It looks like I've posted the above message twice, sorry for