Feature #10592
closedTry to get rid of ZIP error workaround
100%
Description
We made this workaround to handle so called ZIP errors, i.e. when Bridge should receive archives within requests from Core, but it finds out that they are broken. In this case Core infinitely resends such request and I guess that after a second attempt everything is okay (but I did not evaluate that ever).
Maybe after #10579 we will not need this workaround, but we need to estimate this before.
Updated by Vladimir Gratinskiy about 4 years ago
I've added logging of ZIP errors in Bridge in the branch "feature_10592" (file "logs/ZIP.log").
Updated by Evgeny Novikov about 4 years ago
I merged the branch to master. Let's see on logs...
Updated by Evgeny Novikov about 4 years ago
- Priority changed from Urgent to High
- Target version deleted (
3.0)
I do not see any errors related to broken ZIP archives in Bridge logs after several days of extensive operation. So, let's hope that we identified this ugly issue at last. But let's get rid of the workaround after 3.0 to avoid any unpleasant surprises.
Updated by Evgeny Novikov about 4 years ago
- Related to Feature #6587: Extract class Session to common libraries added
Updated by Evgeny Novikov almost 4 years ago
- Target version set to 3.1
It seems that it's time to get rid of this workaround at last.
Updated by Evgeny Novikov almost 4 years ago
- Category set to Bridge
- Assignee changed from Evgeny Novikov to Vladimir Gratinskiy
I got rid of this workaround in Core and Scheduler in branch get-rid-of-zip-error-workaround. Indeed, most work for this issue is in Bridge
Updated by Vladimir Gratinskiy almost 4 years ago
- Due date set to 03/01/2021
- Category deleted (
Bridge) - Status changed from New to Resolved
- Assignee changed from Vladimir Gratinskiy to Evgeny Novikov
- % Done changed from 0 to 100
Implemented in bridge-10592.
Updated by Vladimir Gratinskiy almost 4 years ago
- Category set to Bridge
- Assignee changed from Evgeny Novikov to Vladimir Gratinskiy
Updated by Evgeny Novikov almost 4 years ago
- Status changed from Resolved to Closed
The branch passed tests, so, there is no more ZIP error workaround! Let's hope that nobody will need it anymore.