trafodion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Selva Govindarajan <selva.govindara...@esgyn.com>
Subject RE: FYI - Trafodion check tests are failing
Date Fri, 08 Jul 2016 14:12:59 GMT
Though PR 584 took care of the check PR issues, it could have introduced
some issues for ODBC 3.0 application. A follow up JIRA
https://issues.apache.org/jira/browse/TRAFODION-2100 has been created to
take care of the new issue.

See https://github.com/apache/incubator-trafodion/pull/584 for details of
the conversation.

It is also found that Trafodion ships its own ODBC driver manager in unix.
Does anyone know what is the need for this driver manager?

Selva

-----Original Message-----
From: Suresh Subbiah [mailto:suresh.subbiah60@gmail.com]
Sent: Friday, July 8, 2016 4:57 AM
To: dev@trafodion.incubator.apache.org
Subject: Re: FYI - Trafodion check tests are failing

Hi

PR 584 was merged yesterday. Check tests pass for other PRs now (such as
576). I have kicked off a retest for all PRs in the queue that had a failed
check test.

Thanks
Suresh

On Fri, Jul 8, 2016 at 6:18 AM, Jin, Jian (Seth) <Jian.jin@esgyn.cn> wrote:

> Another fix came in;
> https://github.com/apache/incubator-trafodion/pull/584
>
> Do we still have issue for test failure?
>
> Br,
>
> Seth
>
> -----Original Message-----
> From: Dave Birdsall [mailto:dave.birdsall@esgyn.com]
> Sent: 2016年7月7日 23:52
> To: dev@trafodion.incubator.apache.org
> Subject: RE: FYI - Trafodion check tests are failing
>
> Hi,
>
> If it is simple to back out the change, that's fine.
>
> Another approach is to disable the failing test, and file a JIRA to
> cover that.
>
> Dave
>
> -----Original Message-----
> From: Suresh Subbiah [mailto:suresh.subbiah60@gmail.com]
> Sent: Wednesday, July 6, 2016 5:12 PM
> To: dev@trafodion.incubator.apache.org
> Subject: Re: FYI - Trafodion check tests are failing
>
> Hi,
>
> Sorry for my mistake in merging the PR prematurely.. I did not notice
> that the check test had failed. The change seemed safe.
> Will wait a day for change owner to submit a new change to either
> backout previous change or fix this issue.
> At that point I can try to submit a negative of this PR to get the
> code back to the old state.
>
> Sorry for being careless.
>
> Thanks
> Suresh
>
> On Wed, Jul 6, 2016 at 5:24 PM, Sandhya Sundaresan <
> sandhya.sundaresan@esgyn.com> wrote:
>
> > Sorry - I think I was trying to say reopen the JIRA -306.  Perhaps
> > we jsutneed to undo the change done for that and addresss it
> > differently.
> > But will leave that decision o the JIRA owner - whatever is the
> > quickest to get back on track.
> > Sandhya
> >
> > -----Original Message-----
> > From: Dave Birdsall [mailto:dave.birdsall@esgyn.com]
> > Sent: Wednesday, July 6, 2016 3:21 PM
> > To: dev@trafodion.incubator.apache.org
> > Subject: RE: FYI - Trafodion check tests are failing
> >
> > Hi,
> >
> > There's no such thing as re-opening a PR to my knowledge. The PR has
> > already been merged.
> >
> > Instead, a new PR with the fix must be submitted.
> >
> > Dave
> >
> > -----Original Message-----
> > From: Sandhya Sundaresan [mailto:sandhya.sundaresan@esgyn.com]
> > Sent: Wednesday, July 6, 2016 3:17 PM
> > To: dev@trafodion.incubator.apache.org
> > Subject: FYI - Trafodion check tests are failing
> >
> > Hi all,
> >
> > Since this PR  merge *“[TRAFODION-306] PR-575 LP Bug: 1323868 - ODBC
> > SQLGetTypeInfo api - all datatypes being returned as BIGINT*”,
> > Trafodion Check Tests have been failing.
> >
> > We need to make sure these tests pass at a minimum before merging in
> > changes. The PR will be reopened soon and a comment has been added
> > to it – a fix will be needed ASAP.
> >
> >
> >
> > For the other failures,
> >
> > ·         executor/TEST022 is being addressed. It was caused by PR 564
> > *“**[TRAFODION-1676]support
> > better runtime error message when a SQL function meet fital error”*
> >
> > ·         The hive failures will be addressed in a PR 574 that is  in
> > the
> > pipeline and soon to be merged.
> >
> > ·         Executor test  hang in cdh is a bit tricky and needs more
> > investigation by TM team.
> >
> >
> >
> > The Check Tests do need to get addressed ASAP though (
> > https://github.com/apache/incubator-trafodion/pull/575 ).
> >
> >
> >
> > Thanks
> >
> > Sandhya
> >
>

Mime
View raw message