Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

[TRAFODION-2701] The maxlength for LargeInt was fixed to 8 #1195

Merged
merged 2 commits into from Aug 17, 2017

Conversation

Weixin-Xu
Copy link
Contributor

Snprintf using sizeof(__int64) to set the max length of binding char to bigint with 8 chars.
It is too short for bigint.

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1982/

@Traf-Jenkins
Copy link

@Traf-Jenkins
Copy link

Can one of the admins verify this patch?

@ghost
Copy link

ghost commented Aug 4, 2017

Jenkins, retest.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1987/

@Traf-Jenkins
Copy link

@Weixin-Xu
Copy link
Contributor Author

Jenkins, retest.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1989/

@Traf-Jenkins
Copy link

@Weixin-Xu
Copy link
Contributor Author

Jenkins, retest.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1991/

@Traf-Jenkins
Copy link

@ghost
Copy link

ghost commented Aug 6, 2017

Jenkins, retest.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1992/

@Traf-Jenkins
Copy link

@Weixin-Xu
Copy link
Contributor Author

Jenkins, retest.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/1994/

@Traf-Jenkins
Copy link

@DaveBirdsall
Copy link
Contributor

The JDBC tests seem to be failing consistently. Perhaps that needs to be debugged before we can merge this change?

@DaveBirdsall
Copy link
Contributor

jenkins, retest

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2017/

@Traf-Jenkins
Copy link

@DaveBirdsall
Copy link
Contributor

The test failure is due to a nowait statement limit problem in the JDBC tests. This particular change is not the cause of that. I am merging this change in spite of the failure.

@asfgit asfgit merged commit 6a170de into apache:master Aug 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants