Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

updating takes forever when merging settings #89

Closed
mandric opened this issue Jul 14, 2014 · 2 comments
Closed

updating takes forever when merging settings #89

mandric opened this issue Jul 14, 2014 · 2 comments

Comments

@mandric
Copy link
Member

mandric commented Jul 14, 2014

we are still GET'ing and POST'ing the entire design doc client side to merge settings when updating an app.

screen shot 2014-07-14 at 4 00 51 pm

screen shot 2014-07-14 at 4 00 35 pm
@mandric
Copy link
Member Author

mandric commented Jul 14, 2014

fixes here should also get merged into #85

@mandric mandric mentioned this issue Sep 26, 2014
Closed
@ghost
Copy link

ghost commented Sep 30, 2014

Just to confirm: we believe this to only be present when a slow connection is being used. Essentially, this issue is asking for a more efficient update and/or settings-merge method, so that high-latency/low-throughput connections don't time out and leave the system in an inconsistent state.

The inconsistent state problem is filed as #90.

mandric pushed a commit that referenced this issue Oct 9, 2014
update dashboard core to use app_settings package when available. #89
64d68f8 
Only fetches entire ddoc when app_settings paths are not found.

One fetch of a ddoc is still required right now during settings
migration.  The settings migration path is specified in the `kanso` or
`couchapp` property of the ddoc. We should consider extending the
app_settings module to allow for querying abitrary properties on the
ddoc.
mandric pushed a commit that referenced this issue Oct 14, 2014
refactored app update in dashboard core #89
b7d69c0 
made ddoc fetch during migration optional.

added `has_app_settings_support` function that does a HEAD
requests.  This adds a few additional requests during install
but makes code cleaner.

`migrate_app_settings` function ddoc parameter was removed, is was never
really required in the first place, all we need is the migration path
there.
@mandric mandric closed this as completed Oct 25, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mandric