Zend Framework

Use INSERT DELAYED for Zend_Log_Writer_Db

Details

  • Type: Improvement Improvement
  • Status: Postponed Postponed
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.0.0, 1.0.1, 1.0.2, 1.0.3, 1.0.4, 1.5.0RC1, 1.5.0RC2, 1.5.0RC3, 1.5.0, 1.5.0PL1, 1.5.1, 1.5.3, 1.6.0RC2, Next Major Release
  • Fix Version/s: None
  • Component/s: Zend_Log
  • Labels:
    None

Description

The DELAYED option for the INSERT statement is a MySQL extension to standard SQL that is very useful if you have clients that cannot or need not wait for the INSERT to complete. This is a common situation when you use MySQL for logging (...)

Using INSERT DELAYED would improve the speed of Zend_Log_Writer_Db. But you can't check, if the INSERT was successful. So maybe it should be an optional feature.

Issue Links

This issue is related to:
ZF-9484 INSERT DELAYED (mysql adapter) Critical Open

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Nils Caspar added a comment -

Yes, No, Maybe..?

Show
Nils Caspar added a comment - Yes, No, Maybe..?
Hide
Permalink
Wil Sinclair added a comment -

We'll consider this for 1.8. For now I am marking it postponed, which indicates that the feature request is sound, but we won't work on it immediately. Have you considered submitting a patch and a corresponding unit test to get it on to trunk sooner?
Since it adds a few feature, we will have to wait for the next minor release (1.8).

Show
Wil Sinclair added a comment - We'll consider this for 1.8. For now I am marking it postponed, which indicates that the feature request is sound, but we won't work on it immediately. Have you considered submitting a patch and a corresponding unit test to get it on to trunk sooner? Since it adds a few feature, we will have to wait for the next minor release (1.8).
Hide
Permalink
Benjamin Eberlei added a comment -

one has to check for the mysql adapters when this is a mysql only option, which makes it somewhat more compliacated, otherwise it sounds like a good optimization.

Show
Benjamin Eberlei added a comment - one has to check for the mysql adapters when this is a mysql only option, which makes it somewhat more compliacated, otherwise it sounds like a good optimization.
Hide
Permalink
Thomas Weidner added a comment -

Reassigning to component lead

Show
Thomas Weidner added a comment - Reassigning to component lead
Hide
Permalink
Kamil Dziedzic added a comment -

Here is a patch to resolve this issue and #ZF-9484. If you don't like it please write what's wrong with it so I could fix it.

http://www.kamil.klecza.pl/zend/ZF-3107_ZF-9484.patch

Show
Kamil Dziedzic added a comment - Here is a patch to resolve this issue and #ZF-9484. If you don't like it please write what's wrong with it so I could fix it. http://www.kamil.klecza.pl/zend/ZF-3107_ZF-9484.patch
Hide
Permalink
Benoît Durand added a comment -

@Kamil, you must first sign the CLA in order to use your patch.

Show
Benoît Durand added a comment - @Kamil, you must first sign the CLA in order to use your patch.
Hide
Permalink
Kamil Dziedzic added a comment - - edited

Hello again. I've signed CLA and here is updated patch to ZF 1.11.4 and now it also includes changes to one more similar ticket #ZF-7149.

http://framework.zend.com/issues/secure/attachment/13816/ZF-3107_ZF-7149_ZF-9484.patch

I'm waiting for comments.

Show
Kamil Dziedzic added a comment - - edited Hello again. I've signed CLA and here is updated patch to ZF 1.11.4 and now it also includes changes to one more similar ticket #ZF-7149. http://framework.zend.com/issues/secure/attachment/13816/ZF-3107_ZF-7149_ZF-9484.patch I'm waiting for comments.
Hide
Permalink
Benoît Durand added a comment -

Reassign to the maintener of Zend_Db (Ralph)

Show
Benoît Durand added a comment - Reassign to the maintener of Zend_Db (Ralph)

People

Vote (3)
Watch (5)

Dates

  • Created:
    Updated:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Zend Framework. Try JIRA - bug tracking software for your team.