-
Notifications
You must be signed in to change notification settings - Fork 18
NoSQL #1411
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
NoSQL #1411
Conversation
As discussed in meeting:
|
e12de5d
to
93a6840
Compare
|
bc4173d
to
013f0c9
Compare
013f0c9
to
03ddb21
Compare
Codecov ReportAttention: Patch coverage is
❌ Your project check has failed because the head coverage (9%) is below the target coverage (70%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #1411 +/- ##
========================================
- Coverage 12% 9% -2%
========================================
Files 265 252 -13
Lines 7397 7234 -163
========================================
- Hits 819 650 -169
- Misses 6578 6584 +6 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Watch out for the changes made in the restaurant_page_view, might cause some conflicts due to the redesign developments.
Good Point! I think I should wait for redesign to be merge on develop or change the origin branch to ui/redesign |
One model was updated, I don't remember the one. The restaurant? |
Missing feature:
|
Closes #1391 closes #755
Used Object Box, which is not the ideal approach since it require new annotations on models and don't rely on the existing serialization.
Bus stops remain in
sqlite
since they are not working and fix code need to be written.Comparing SQlite and NoSQL
I run a simple experiment to compare the read and initialization times with my personal Exams data (around total 8 exams)
Conclusion
Since we had
9
databases and assuming they all boot up the same time as Exams, sequentially all of them would make171ms
to set up. Base on that, both of the technologies have similar db initialization times for a scenario using all the models (example: homepage with all of possible cards). However, read times were significantly decreased.