Changes between Initial Version and Version 1 of Ticket #27539, comment 7


Ignore:
Timestamp:
Feb 1, 2017, 11:59:09 AM (7 years ago)
Author:
reficul31

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #27539, comment 7

    initial v1  
    1 Skimming over the code. I think the best solution i could think of once the count reaches 9000 we start removing queries from the back of the queue as queries_log is a deque . But I do have a question. Should the method to remove the queries_log be put in BaseDatabaseWrapper or can we just remove them manually in the CaptureQueriesContext? Or we could just throw an error?? But the problem will recurr and the following tests will also start throwing the same error.
    2 Replying to [comment:6 Aymeric Augustin]:
    3 > The test framework should clear `connection.queries` between each test.
    4 Do you want to clear all the queries or can it just be like a pop function where as a query comes in another one is deleted?
Back to Top