Page MenuHomePhabricator
Differential D11702

Remove unecessary SQLite initialization from CommHybrid
ClosedPublic
Actions

Authored by marcin on Apr 19 2024, 9:48 AM.
Tags
None
Referenced Files
F2018923: D11702.id39290.diff
Sun, Jun 16, 2:58 AM
Unknown Object (File)
Sat, Jun 15, 12:18 AM
Unknown Object (File)
Fri, Jun 14, 7:30 AM
Unknown Object (File)
Thu, Jun 13, 8:22 PM
Unknown Object (File)
Thu, Jun 13, 6:19 PM
Unknown Object (File)
Sat, May 25, 8:03 PM
Unknown Object (File)
Fri, May 24, 1:25 AM
Unknown Object (File)
Fri, May 24, 1:24 AM
View All 24 Files
Subscribers
None

Details

Reviewers
kamil
ashoat
tomek
Commits
rCOMMfaf8cda7751a: Remove unecessary SQLite initialization from CommHybrid
Summary

This differential removes unnecessary and potentially buggy SQLite initialization from CommHybrid. The context is here:
https://linear.app/comm/issue/ENG-7700/reconsider-double-sqlitequeryexecutorinitialize-calls-on-android

Test Plan

Uninstall and reinstall Android app a couple of times. Ensure each time app works correctly.

Diff Detail

Repository
rCOMM Comm
Branch
marcin/eng-7700
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

marcin created this revision.Apr 19 2024, 9:48 AM
marcin added 1 blocking reviewer(s): kamil.Apr 19 2024, 9:49 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Apr 19 2024, 9:58 AM
Harbormaster failed remote builds in B28331: Diff 39290!
Harbormaster completed remote builds in B28331: Diff 39290.Apr 19 2024, 10:08 AM
marcin requested review of this revision.Apr 19 2024, 10:08 AM
ashoat accepted this revision.Apr 19 2024, 12:02 PM

Nice to clean up the additionalParameters stuff

kamil accepted this revision.Apr 22 2024, 3:40 AM
This revision is now accepted and ready to land.Apr 22 2024, 3:40 AM
marcin updated this revision to Diff 39337.Apr 22 2024, 5:09 AM

Rebase before landing

Harbormaster completed remote builds in B28353: Diff 39337.Apr 22 2024, 5:24 AM
Closed by commit rCOMMfaf8cda7751a: Remove unecessary SQLite initialization from CommHybrid (authored by marcin). · Explain WhyApr 22 2024, 5:33 AM
This revision was automatically updated to reflect the committed changes.
marcin added a commit: rCOMMfaf8cda7751a: Remove unecessary SQLite initialization from CommHybrid.

Revision Contents
Changeset List

PathSize
native/
android/
app/
src/
cpp/
12 lines
main/
java/
app/
comm/
android/
fbjni/
12 lines

Diff 39290

View Options

native/android/app/src/cpp/jsiInstaller.cpp

Loading...
View Options

native/android/app/src/main/java/app/comm/android/fbjni/CommHybrid.java

Loading...