I have merged the PR 1335. I will trigger the retest of all failed PR due to this issue. Thanks Gong gongpen...@innfotech.com for taking care of this issue.
Selva -----Original Message----- From: Selva Govindarajan [mailto:selva.govindara...@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.sha...@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
