Skip to content
This repository has been archived by the owner on Oct 28, 2020. It is now read-only.

Force reindexing of requests after defining them #19

Merged
merged 3 commits into from
Apr 17, 2015

Conversation

aenario
Copy link
Contributor

@aenario aenario commented Apr 17, 2015

Allow apps to monitor couchdb reindexing of their requests and display an helpful message instead index pages timing out while the app try to load data.

@jsilvestre
Copy link
Contributor

Are you sure it's the way to go? I mean VS forcing reindexation in the DS. Could you expand on your thought process?

@aenario
Copy link
Contributor Author

aenario commented Apr 17, 2015

There is 2 issues :
1 - At view creation / update
The best way would be to make this part of the installation process. However, this implies some new API & communication between DS, controller & app. Because the controller knows nothing about the app views and the DS doesn't know how many views an app is creating.
What I did in this PR is to allow an app to know if its views are ready or not so it can display a "Please wait for reindexing" message instead.

2 - After inserting x000 messages
The DS should reindex all views.

@jsilvestre
Copy link
Contributor

Got it, thanks!

@frankrousseau
Copy link
Contributor

Tests are broken by your changes.

@aenario
Copy link
Contributor Author

aenario commented Apr 17, 2015

Oups, some crazy maniac put linting in tests :)

frankrousseau pushed a commit that referenced this pull request Apr 17, 2015
Force reindexing of requests after defining them
@frankrousseau frankrousseau merged commit f865143 into cozy:master Apr 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants