[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

How about making use of
A post-commit hooks in git seems to be an easy way to achieve this.
Unfortunately, it would require that each fresh clone of the repository has
a one-time command run to install the hook.

Michael Mior

Le lun. 19 nov. 2018 à 14:49, Julian Hyde <jhyde@xxxxxxxxxx> a écrit :

> > On Nov 19, 2018, at 11:19 AM, Vladimir Sitnikov <
> sitnikov.vladimir@xxxxxxxxx> wrote:
> >
> > 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.
> Yes, we should do that.
> However there are things that automation could never achieve, so let’s
> continue to talk about those, also.
> > 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.
> I agree. The commit didn’t seem quite perfect to me either. However, it
> seemed to be progress. Log an Avatica JIRA if you have ideas for how to
> improve it further. Since it will be in Avatica it will take a while to
> bubble through the release cycle.
> Julian