Skip to content
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

Part1: Forward notifications between StorIOSQLite v1 and v3 #876

Closed
wants to merge 7 commits into from

Conversation

nikitin-da
Copy link
Collaborator

Part of #875

@nikitin-da nikitin-da added this to the v3.0.1 milestone Dec 24, 2017
@nikitin-da nikitin-da self-assigned this Dec 24, 2017
@codecov-io
Copy link

codecov-io commented Dec 24, 2017

Codecov Report

Merging #876 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #876      +/-   ##
============================================
+ Coverage     97.99%   98.01%   +0.01%     
- Complexity      815      819       +4     
============================================
  Files            98       99       +1     
  Lines          2787     2815      +28     
  Branches        308      310       +2     
============================================
+ Hits           2731     2759      +28     
  Misses           27       27              
  Partials         29       29
Impacted Files Coverage Δ Complexity Δ
...h/storio3/sqlite/interop1to3/StorIOSQLite1To3.java 100% <100%> (ø) 4 <4> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b12beef...b6457f1. Read the comment docs.

@nikitin-da nikitin-da changed the title Forward notifications between StorIOSQLite v1 and v3 Part1: Forward notifications between StorIOSQLite v1 and v3 Dec 30, 2017
@geralt-encore
Copy link
Collaborator

@nikitin-da @artem-zinnatullin Do you think that it is a good idea to have it? I mean a migration to a newer version shouldn't be hard and it is a lot of code to maintain and care about.

@nikitin-da
Copy link
Collaborator Author

@geralt-encore happy new year man! 🎄 🍾 🍊

@nikitin-da
Copy link
Collaborator Author

a migration to a newer version shouldn't be hard

I'm totally agree that migration of storio itself is not a big deal and can be done by single pull request.
But most likely we have to migrate to RxJava2 as well and here we can meet some problems if we use storio-rx widely.
My current project has no separate repository that can be fast converted to storio3 but keep outer interface on rx1 by RxJavaInterop. I guess I'm not alone in this problem and many projects will have to migrate storio and rx stepwise.

it is a lot of code to maintain and care about

Firstly I thought that some gist with notifications sharing will be enough. But lately realized that I have to migrate changes and mapping as well. It is not feet to gist and requires tests..
What about code maintaining I think to prevent induced changes in this module we may use fixed storio 3.0.0 version here instead of subproject dependency.
And than it will lead to separate interop version (because otherwise interop version will outrun storio itself).

Or maybe we should create another repository and publish it independently like it do @akarnokd.
@artem-zinnatullin any thoughts?

@geralt-encore
Copy link
Collaborator

@geralt-encore happy new year man! 🎄 🍾 🍊
Thanks, mate! Happy new year to you, too!

I see what problem you are trying to solve and it feels more like RxJava migration related than StorIO related. Also, it might get out of hand pretty fast. Imagine that we would release storio4 and 3 more migrations are needed.

I like the idea of keeping it in separate repo and publishing as an independent artefact.

@nikitin-da nikitin-da closed this May 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants