[INP-1572] Change semaphores to allow concurrent editing of different DB records - In-Portal Issue Tracker

XML

Word

Printable

Change Log Group:
Fixed
Change Log Message:
Saving changes to different DB records from same table wasn't possible due when semaphore protection.
Story Points:
1
Copy Issue Key:
Patch Instructions:
Patches must be submitted through Phabricator.
- To submit patch via Command Line use Patches Workflow (via Arcanist) tutorial
- To submit patch via Web Interface use Patches Workflow (via Web Interface) tutorial

Plan:

add "MainIDs" column (text, null) to "Semaphores" table - 0.2h
when creating a semaphore ("kTempTablesHandler::SaveEdit" method) store main table IDs comma-separated form in the new "Semaphores.MainIDs" table - 0.3h
when checking for existing semaphores exclude semaphore, when: - 1h
- the copied IDs (will go into "MainIDs" column if semaphore is created) do not overlap with IDs in the semaphore
- the "MainIDs" column of the semaphore has "0" value (in main temp table new created record always has 0 as its ID due missing auto-increment column)

Quote: 1.5h*1.4=2h

blocks

INP-1780 Sync "Semaphores" table structure between clean install and upgrade

implemented in

D237 INP-1572 - Change semaphores to allow concurrent editing of different DB records

Estimated:

Remaining:

0.2h

Logged:

1.8h