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.

Sign up for GitHub

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

Jump to bottom

Update to DBConnection 1.2 #196

Open

fishcakez wants to merge 6 commits into xerions:master from fishcakez:jf-db_conn-1.2

Contributor

fishcakez commented Aug 11, 2017 •

edited

DBConnection 1.2 is not released yet but this PR tracks progress.

Use database transaction status to prevent transaction errors
Support new low level cursor API with per fetch max rows
Handle execute errors while using cursors
Test that transaction status is updated for more query result packets

coveralls commented Aug 11, 2017 •

edited

Coverage increased (+0.3%) to 66.033% when pulling 1d3b6a2 on fishcakez:jf-db_conn-1.2 into c654c88 on xerions:master.

coveralls commented Aug 12, 2017

Coverage increased (+3.4%) to 69.108% when pulling 789c9a6 on fishcakez:jf-db_conn-1.2 into c654c88 on xerions:master.

1 similar comment

coveralls commented Aug 12, 2017

Coverage increased (+3.4%) to 69.108% when pulling 789c9a6 on fishcakez:jf-db_conn-1.2 into c654c88 on xerions:master.

fishcakez force-pushed the jf-db_conn-1.2 branch from 789c9a6 to c570005 Compare

August 12, 2017 23:32

Contributor Author

fishcakez commented Aug 12, 2017

Still a bit more work to be done here: more tests and configuring max rows at per fetch level.

coveralls commented Aug 12, 2017 •

edited

Coverage increased (+3.8%) to 69.507% when pulling c570005 on fishcakez:jf-db_conn-1.2 into c654c88 on xerions:master.

coveralls commented Aug 13, 2017 •

edited

Coverage increased (+4.2%) to 69.973% when pulling 696d037 on fishcakez:jf-db_conn-1.2 into c654c88 on xerions:master.

1 similar comment

coveralls commented Aug 13, 2017

Coverage increased (+4.2%) to 69.973% when pulling 696d037 on fishcakez:jf-db_conn-1.2 into c654c88 on xerions:master.

michalmuskala mentioned this pull request

New Release elixir-ecto/db_connection#92

Closed

fishcakez mentioned this pull request

GenStage and Ecto streams? elixir-lang/gen_stage#150

Closed

fishcakez added 6 commits

November 26, 2017 14:45


          Use database transaction status to prevent transaction errors

c813c4a


          Use one text statement for all queries

75d1482


          Support explicit cursor API

bba273d


          Remove reserved query name errors

d088b77


          Add max rows option to each cursor fetch

64d51fb


          Raise DBConnection.ConnectionError on connection error

cf37d53

fishcakez force-pushed the jf-db_conn-1.2 branch from 696d037 to cf37d53 Compare

November 26, 2017 23:25

coveralls commented Nov 26, 2017 •

edited

Coverage increased (+3.4%) to 71.04% when pulling cf37d53 on fishcakez:jf-db_conn-1.2 into e9eb29f on xerions:master.

6 similar comments

coveralls commented Nov 26, 2017

Coverage increased (+3.4%) to 71.04% when pulling cf37d53 on fishcakez:jf-db_conn-1.2 into e9eb29f on xerions:master.

coveralls commented Nov 26, 2017 •

edited

Coverage increased (+3.4%) to 71.04% when pulling cf37d53 on fishcakez:jf-db_conn-1.2 into e9eb29f on xerions:master.

coveralls commented Nov 26, 2017

Coverage increased (+3.4%) to 71.04% when pulling cf37d53 on fishcakez:jf-db_conn-1.2 into e9eb29f on xerions:master.

coveralls commented Nov 26, 2017

Coverage increased (+3.4%) to 71.04% when pulling cf37d53 on fishcakez:jf-db_conn-1.2 into e9eb29f on xerions:master.

coveralls commented Nov 26, 2017

Coverage increased (+3.4%) to 71.04% when pulling cf37d53 on fishcakez:jf-db_conn-1.2 into e9eb29f on xerions:master.

coveralls commented Nov 26, 2017

Coverage increased (+3.4%) to 71.04% when pulling cf37d53 on fishcakez:jf-db_conn-1.2 into e9eb29f on xerions:master.

josevalim mentioned this pull request

Update to DBConnection 2.0 #235

Merged

4 tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment