Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.4] Payload statement at the end of the insert array of a job + test #22343

Merged
merged 1 commit into from
Dec 7, 2017

Conversation

gabomasi
Copy link
Contributor

@gabomasi gabomasi commented Dec 7, 2017

I need to fix this on 5.4 too... #22334
Moreover, I am adding a unit test

Greetings
\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
#22334
Rollback 7c8d3d0

It was opened an issue in laravel-oci8 because of an exception thrown when inserting in the table jobs.

The issue was fixed on 7c8d3d0 but it was changed on a041fb5 for no reason. I created a new one and generated this pull request to solve the problem again.

ORA-24816: Expanded non LONG bind data supplied after actual LONG or LOB column
Cause: A Bind value of length potentially > 4000 bytes follows binding for LOB or LONG.
Action: Re-order the binds so that the LONG bind or LOB binds are all at the end of the bind list.

Like is mentioned above, to solve this is simply needed to put the clob column at the end of the insert statement.

Greetings
\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\

Put the payload column of jobs table to the end of insert statements.
@taylorotwell taylorotwell merged commit 1fa9dac into laravel:5.4 Dec 7, 2017
gabomasi added a commit to gabomasi/framework that referenced this pull request Dec 7, 2017
@gabomasi gabomasi deleted the 5.4 branch December 7, 2017 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants