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

Bug #57702 :: Multiple BLOB fetch fails. #217

Closed
wants to merge 1 commit into from

Conversation

hswong3i
Copy link

Please refer to https://bugs.php.net/bug.php?id=57702 for more information.

P.S. May someone help to contribute the php-level test case for this issue?

Sorry that since this bug will ALWAYS hit during installation when develop with Drupal 7.x pdo_oci driver, but very hard to reproduce its trigger with simple test cases...

Hope someone can give a hand for it ;-)

@lstrojny
Copy link
Contributor

Could you add a test for this bug as well. Otherwise it looks good.

@hswong3i
Copy link
Author

Sorry that I try to reproduce this bug with a test case once before, but really no luck.

It only happen when running a relatively long operation, and then hit the bug suddenly:

  • When hitting the bug, is that we are running a specific query? No, the query already run multiple times in same instance before hitting the bug
  • Any special syntax/logic for the query that hitting the bug? No, just very normal query, which if we are running it individually, bug will never hit; we only hit when running a relatively long operation

Therefore the difficult of providing the test case is: I can't figure out the common factor why bug is being triggered. From other point of view, yes this bug will always trigger during Drupal 7.x installation, but not means we should include ALL query that running into the test case, isn't it?

Since this patch is not originally propose by me, what I could comment on it are:

  • From logical point of view, it should works
  • From practical point of view, it also works as expected
  • From test case point of view, it is difficult to provide supporting :(

@laruence
Copy link
Member

this is already fixed in 4fcf4e0

sorry didn't see this PR before

@php-pulls
Copy link

Comment on behalf of laruence at php.net:

fixed

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.

4 participants