recursive read side jdbc handler

[AntonioYuen]

Recursive read side jdbc handler with exponential backoff
Decouples our jdbc processor retry from Akka

resolves #318

[Tochemey]

Hey @AntonioYuen I don't think you need to implement this. You can just see how it is set because java grpc does have that feature.

• https://stackoverflow.com/questions/55428539/grpc-java-setting-enableretry-and-maxretryattempt-is-not-retrying
• https://github.com/grpc/grpc-java/blob/master/core/src/main/java/io/grpc/internal/ExponentialBackoffPolicy.java
• https://github.com/grpc/grpc/blob/master/doc/connection-backoff.md

[Tochemey]

Please can we avoid this?

[AntonioYuen]

Hey @AntonioYuen I don't think you need to implement this. You can just see how it is set because java grpc does have that feature.

• https://stackoverflow.com/questions/55428539/grpc-java-setting-enableretry-and-maxretryattempt-is-not-retrying
• https://github.com/grpc/grpc-java/blob/master/core/src/main/java/io/grpc/internal/ExponentialBackoffPolicy.java
• https://github.com/grpc/grpc/blob/master/doc/connection-backoff.md

Will look into it

[Tochemey]

Please can we get a clear details on the use case we are solving? Also open an issue before this PR.

[Tochemey]

Please can we get a clear details on the use case we are solving? Also open an issue before this PR.

Never mind. @zenyui has explained to me. Kindly add it to the PR description.

[codecov]

Codecov Report

Merging #281 (be9e672) into master (5121c32) will decrease coverage by 0.38%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #281      +/-   ##
==========================================
- Coverage   77.91%   77.52%   -0.39%     
==========================================
  Files          42       42              
  Lines         797      801       +4     
  Branches       20       21       +1     
==========================================
  Hits          621      621              
- Misses        176      180       +4     

Impacted Files	Coverage Δ
...ly/chiefofstate/readside/ReadSideJdbcHandler.scala	0.00% <0.00%> (ø)
...mely/chiefofstate/readside/ReadSideProcessor.scala	0.00% <0.00%> (ø)

---

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 5121c32...be9e672. Read the comment docs.

got the explanation needed

[Tochemey]

@AntonioYuen I think we can get this through.

[Tochemey]

@AntonioYuen it seems this PR is outdated and need some love.

[AntonioYuen]

@AntonioYuen it seems this PR is outdated and need some love.

I will give it some love this week.