[MAGNOLIA-4090] VirtualUriManager should be reimplemented to extend ModuleConfigurationObservingManager Created: 22/Sep/11 Updated: 17/Feb/16 Resolved: 12/Feb/16 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | None |
| Fix Version/s: | 5.4.5 |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Tobias Mattsson | Assignee: | Tobias Mattsson |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | 0.25h | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Release notes required: |
Yes
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||||||||||||||||||||||||||
| Sprint: | Kromeriz 30 | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Story Points: | 3 | ||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
A restart will then no longer be required when adding the virtualURIMappings node, It will allow us to fix the concurrency issues when configuration is reloaded, We should also look at adding support for folders as described in |
| Comments |
| Comment by Mikaël Geljić [ 21/Jan/16 ] |
|
linking to |
| Comment by Roman Kovařík [ 22/Jan/16 ] |
|
mgeljic The above comment/link probably belongs to |
| Comment by Mikaël Geljić [ 22/Jan/16 ] |
|
Well here it's about our "client" usage of registries/managers, specifically about virtualUriMappings; At first when I saw the PR I thought we were taking the step towards the AbstractRegistry already; here now it seems we're stopping half-way through, which is fine, but eventually 6415 should probably supersede this. |
| Comment by Roman Kovařík [ 22/Jan/16 ] |
|
Reopened: |