Project

General

Profile

Actions

Feature #47712

closed

Refactor Locking

Added by Daniel Hürtgen almost 11 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Backend API
Target version:
Start date:
2013-02-08
Due date:
% Done:

100%

Estimated time:
PHP Version:
5.5
Tags:
Complexity:
hard
Sprint Focus:
Remote Sprint

Description

-- to be continued --


Related issues 4 (0 open4 closed)

Related to TYPO3 Core - Bug #55294: Under heavy load, two request can simultaneously write to same cache file - should use lockingClosedBenni Mack2014-01-24

Actions
Related to TYPO3 Core - Bug #55099: PHP Warnings after clearing configuration cache in BEClosed2014-01-17

Actions
Related to TYPO3 Core - Bug #66503: Core: Error handler (FE): PHP Warning: sem_get(): failed for key 0xbaa3533: No space left on deviceClosedMarkus Klein2015-04-20

Actions
Precedes TYPO3 Core - Bug #66411: New locking API exceptions too genericClosedMathias Brodala2015-04-14

Actions
Actions #1

Updated by Gerrit Code Review almost 11 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20681

Actions #2

Updated by Gerrit Code Review over 10 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20681

Actions #3

Updated by Gerrit Code Review over 10 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20681

Actions #4

Updated by Ernesto Baschny over 10 years ago

  • Category changed from Code Cleanup to Backend API

Will work continue on this front? Is there a concept plan or some documentation on the end goal of this refactoring?

This is more than a "Code Cleanup", it introduces a new Locking API. So it should aim to replace the current "Locker" class. So missing still is:

1) a "flock" locking mechanism
2) make it used by the Core, making sure the old "Locker" class is not used anymore
3) deprecate the Locker class

Is it realistic that we reach the goal of finishing this API before feature freeze (which would be next week but will most probably be postponed) and that we reach the goal of "replacing the usage of Locker" before the final release?

Actions #5

Updated by Ernesto Baschny over 10 years ago

  • Target version deleted (6.2.0)
Actions #6

Updated by Christian Kuhn over 9 years ago

  • Status changed from Under Review to Rejected

While the work on this is highly appreciated and the pending patch was a good step into the right direction, it was pending as WIP in gerrit for a year now without anyone really working on it. The patch was abandoned for the time being. Meanwhile, a blueprint for locking evolved at http://wiki.typo3.org/Blueprints/LockingForCaches and the work here is mentioned there for anyone to pick up.

This issue here will be set to "rejected" for now, since further works needs to be aligned and decided in the blueprint, and after that realized in smaller and new issues.

Actions #7

Updated by Markus Klein about 9 years ago

  • Status changed from Rejected to Accepted
  • Assignee changed from Daniel Hürtgen to Markus Klein
  • Target version set to 7.2 (Frontend)
  • PHP Version changed from 5.3 to 5.5
  • Complexity changed from nightmare to hard
Actions #8

Updated by Gerrit Code Review about 9 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #9

Updated by Gerrit Code Review about 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #10

Updated by Gerrit Code Review about 9 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #11

Updated by Gerrit Code Review about 9 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #12

Updated by Gerrit Code Review about 9 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #13

Updated by Gerrit Code Review about 9 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #14

Updated by Gerrit Code Review about 9 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #15

Updated by Gerrit Code Review about 9 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #16

Updated by Gerrit Code Review about 9 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #17

Updated by Markus Klein about 9 years ago

  • Sprint Focus set to Remote Sprint
Actions #18

Updated by Gerrit Code Review about 9 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #19

Updated by Gerrit Code Review about 9 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #20

Updated by Gerrit Code Review about 9 years ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #21

Updated by Gerrit Code Review about 9 years ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #22

Updated by Gerrit Code Review about 9 years ago

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #23

Updated by Gerrit Code Review about 9 years ago

Patch set 15 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #24

Updated by Gerrit Code Review about 9 years ago

Patch set 16 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #25

Updated by Gerrit Code Review about 9 years ago

Patch set 17 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #26

Updated by Gerrit Code Review about 9 years ago

Patch set 18 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #27

Updated by Gerrit Code Review about 9 years ago

Patch set 19 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37700

Actions #28

Updated by Markus Klein about 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 40 to 100
Actions #29

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF