You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@trafodion.apache.org by Anoop Sharma <an...@esgyn.com> on 2017/12/11 05:11:10 UTC
trafodion merge is causing all checktests to fail
Recently a PR(1283) was merged into trafodion branch which is now causing
all jdbc checktests as well as traf daily Jenkins build jdbc tests to fail.
Another PR(1320) has been created to track the failure caused by the merge
of the original PR.
Until PR 1320 is fixed, traf jdbc tests should be disabled otherwise no one
will be able to deliver into trafodion branch.
Related note. Any PR(like 1283) that have not passed checktests should not be
merged into trafodion branch unless there is a reason to do so and the merge
would not cause future checktests to fail.
anoop
RE: trafodion merge is causing all checktests to fail
Posted by Anoop Sharma <an...@esgyn.com>.
thanks for getting this resolved.
anoop
-----Original Message-----
From: Selva Govindarajan [mailto:selva.govindarajan@esgyn.com]
Sent: Monday, December 11, 2017 7:01 AM
To: dev@trafodion.incubator.apache.org
Subject: RE: trafodion merge is causing all checktests to fail
I have merged the PR 1335. I will trigger the retest of all failed PR due to this issue. Thanks Gong gongpengju@innfotech.com for taking care of this issue.
Selva
-----Original Message-----
From: Selva Govindarajan [mailto:selva.govindarajan@esgyn.com]
Sent: Monday, December 11, 2017 6:53 AMGong
To: dev@trafodion.incubator.apache.org
Subject: RE: trafodion merge is causing all checktests to fail
I inadvertently merged the PR 1283 overlooking the fact that the check-PR has failed. It looks like this issue has been corrected via https://github.com/apache/incubator-trafodion/pull/1335. This PR has passed the check-PR. I will merge this PR enabling check-PR tests to pass.
Usually, I am diligent in committing the change, in the zest to clean up the long list of pending PRs, I somehow have overlooked when I merged the original PR https://github.com/apache/incubator-trafodion/pull/1283.
I sincerely regret the inconvenience caused to all the contributors.
Selva
-----Original Message-----
From: Anoop Sharma [mailto:anoop.sharma@esgyn.com]
Sent: Sunday, December 10, 2017 9:11 PM
To: dev@trafodion.incubator.apache.org
Subject: trafodion merge is causing all checktests to fail
Recently a PR(1283) was merged into trafodion branch which is now causing all jdbc checktests as well as traf daily Jenkins build jdbc tests to fail.
Another PR(1320) has been created to track the failure caused by the merge of the original PR.
Until PR 1320 is fixed, traf jdbc tests should be disabled otherwise no one will be able to deliver into trafodion branch.
Related note. Any PR(like 1283) that have not passed checktests should not be merged into trafodion branch unless there is a reason to do so and the merge would not cause future checktests to fail.
anoop
RE: trafodion merge is causing all checktests to fail
Posted by Selva Govindarajan <se...@esgyn.com>.
I have merged the PR 1335. I will trigger the retest of all failed PR due to this issue. Thanks Gong gongpengju@innfotech.com for taking care of this issue.
Selva
-----Original Message-----
From: Selva Govindarajan [mailto:selva.govindarajan@esgyn.com]
Sent: Monday, December 11, 2017 6:53 AMGong
To: dev@trafodion.incubator.apache.org
Subject: RE: trafodion merge is causing all checktests to fail
I inadvertently merged the PR 1283 overlooking the fact that the check-PR has failed. It looks like this issue has been corrected via https://github.com/apache/incubator-trafodion/pull/1335. This PR has passed the check-PR. I will merge this PR enabling check-PR tests to pass.
Usually, I am diligent in committing the change, in the zest to clean up the long list of pending PRs, I somehow have overlooked when I merged the original PR https://github.com/apache/incubator-trafodion/pull/1283.
I sincerely regret the inconvenience caused to all the contributors.
Selva
-----Original Message-----
From: Anoop Sharma [mailto:anoop.sharma@esgyn.com]
Sent: Sunday, December 10, 2017 9:11 PM
To: dev@trafodion.incubator.apache.org
Subject: trafodion merge is causing all checktests to fail
Recently a PR(1283) was merged into trafodion branch which is now causing all jdbc checktests as well as traf daily Jenkins build jdbc tests to fail.
Another PR(1320) has been created to track the failure caused by the merge of the original PR.
Until PR 1320 is fixed, traf jdbc tests should be disabled otherwise no one will be able to deliver into trafodion branch.
Related note. Any PR(like 1283) that have not passed checktests should not be merged into trafodion branch unless there is a reason to do so and the merge would not cause future checktests to fail.
anoop
RE: trafodion merge is causing all checktests to fail
Posted by Selva Govindarajan <se...@esgyn.com>.
I inadvertently merged the PR 1283 overlooking the fact that the check-PR has failed. It looks like this issue has been corrected via https://github.com/apache/incubator-trafodion/pull/1335. This PR has passed the check-PR. I will merge this PR enabling check-PR tests to pass.
Usually, I am diligent in committing the change, in the zest to clean up the long list of pending PRs, I somehow have overlooked when I merged the original PR https://github.com/apache/incubator-trafodion/pull/1283.
I sincerely regret the inconvenience caused to all the contributors.
Selva
-----Original Message-----
From: Anoop Sharma [mailto:anoop.sharma@esgyn.com]
Sent: Sunday, December 10, 2017 9:11 PM
To: dev@trafodion.incubator.apache.org
Subject: trafodion merge is causing all checktests to fail
Recently a PR(1283) was merged into trafodion branch which is now causing all jdbc checktests as well as traf daily Jenkins build jdbc tests to fail.
Another PR(1320) has been created to track the failure caused by the merge of the original PR.
Until PR 1320 is fixed, traf jdbc tests should be disabled otherwise no one will be able to deliver into trafodion branch.
Related note. Any PR(like 1283) that have not passed checktests should not be merged into trafodion branch unless there is a reason to do so and the merge would not cause future checktests to fail.
anoop