calcite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jacques Nadeau <jacq...@apache.org>
Subject Re: [VOTE] Release apache-calcite-1.0.0-incubating (release candidate 3)
Date Tue, 03 Feb 2015 20:27:09 GMT
Running mvn clean install on this release artifact, I got out of memory
exception.  Running again to see if transient.  Any others seeing this?

On Tue, Feb 3, 2015 at 11:10 AM, James Taylor <jamestaylor@apache.org>
wrote:

> +1. Ran the same tests as before and everything worked well.
>
> Thanks,
> James
>
> On Mon, Feb 2, 2015 at 11:05 AM, Julian Hyde <julianhyde@gmail.com> wrote:
> > Yes, CALCITE-582 is fixed. It says so in the commit log, the release
> notes and the RC3 vote email.
> >
> > Re Jacques’ comments: I have not marked it fixed because I have not
> merged it to master yet. When incorporating Vladimir’s patch I had to make
> some other changes to make ScannableTableTest.testSimple pass.
> >
> > I have not had chance to look at CALCITE-580 yet. I do not think it is
> fixed, and I do not consider it a show-stopper for the release.
> >
> > Julian
> >
> > Chin Wei Low is running into a different issue, probably
> > On Feb 1, 2015, at 7:05 PM, Jacques Nadeau <jacques@apache.org> wrote:
> >
> >> I'm actually a bit confused on this.  The branch-1.0 shows that
> CALCITE-582
> >> was merged but I don't see it marked resolved nor a +1.  It also looks
> like
> >> Vladimir was the author but he doesn't like the fix.
> >>
> >> On Sun, Feb 1, 2015 at 5:33 PM, Chin Wei Low <lowchinwei@gmail.com>
> wrote:
> >>
> >>> Hi,
> >>>
> >>> Did issue 582 fixed?
> >>>
> >>> Testing with query like this: select a from abc
> >>> But, it throw the exception.
> >>> Exception in thread "main" java.lang.ClassCastException:
> >>> [Ljava.lang.Object; incompatible with java.lang.Integer
> >>> at
> >>>
> >>>
> org.apache.calcite.avatica.util.AbstractCursor$IntAccessor.getInt(AbstractCursor.java:460)
> >>> at
> >>>
> >>>
> org.apache.calcite.avatica.AvaticaResultSet.getInt(AvaticaResultSet.java:311)
> >>>
> >>> On Mon, Feb 2, 2015 at 12:37 AM, Vladimir Sitnikov <
> >>> sitnikov.vladimir@gmail.com> wrote:
> >>>
> >>>>> I needed to fix the bug and get a release out with the minimum
> >>>> of disruption
> >>>>
> >>>> I see, however I raised the question since this area clearly is
> >>>> related to public API.
> >>>> I do not like "just fix the bug" approach without agreed APIs.
> >>>>
> >>>>> CALCITE-558. Please get involved
> >>>>
> >>>> Well, I try to block/prevent one more non-required convention to
> appear.
> >>>>
> >>>> Vladimir
> >>>>
> >>>
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message