Update pgx v3 to v4 #32
Conversation
winebarrel
force-pushed
the
pgx-v3
branch
3 times, most recently
from
1a6a451
to
fcd9b01
Compare
| if err != nil { | ||
| t.Fatal(err) | ||
| } | ||
| // using stdlib, it's difficult to open max conn from the begining | ||
| db := sql.OpenDB(connector) | ||
| // using stdlib, it's difficult to open max conn from the beginning |
| @@ -6,6 +6,10 @@ test: vet | |||
| vet: | |||
There was a problem hiding this comment.
go vet ./... だけであれば、golangci-lint で govet を enable にしてしまってもいいかもしれません。自作の vettool を噛ませる場合は go vet が単独であってもよさそうです。
There was a problem hiding this comment.
どもども。golangci-lintは非標準なので、手元の動作確認としては標準のgo vetの利用が良いかと考えたので、現状のままで行こうかと思います 🙏
| return GetConnectorFromConnStr(u.String()) | ||
| } | ||
|
|
||
| func GetConnectorFromConnStr(connStr string) (driver.Connector, error) { |
There was a problem hiding this comment.
[q] GetConnector -> GetConnectorFromURL -> GetConnectorFromConnStr となっていて、実際には
https://github.com/kanmu/qg/pull/32/files#diff-455862be1b95c56476ae56be62394efde22297b6ae74d7bb372d1db40834d21dR21 で GetConnector() が使われているようですが、わざわざ関数に分けてるのって何でですか？(実装の見通し、他の箇所で connStr をそのまま使いたい箇所がある...etc)
There was a problem hiding this comment.
pgxは基本的にURLをパースしたconfigを使うことを想定しているようです。
ただ、host, port, username, password, dbが別々のparameterで管理されるようなユースケースがあり、都度 urlオブジェクトを組み立てるのが手間であるため（エスケープを考えるとurlオブジェクトは使いたい）
このような形にしています。
|
🙏 |
2999bea からforkして新しくpgx v4の対応を行います。
既存の動作をなるべく変更しない方針で修正しました。
（標準ライブラリのコネクションプールを使用・pgxのprepared statementを使用、等）
APIの呼び出しは変わりましたが本質的な動作は以前と変わっていないと思います。
また修正のマージ後、 2999bea 空のブランチ new-master 新しいmasterに変更します
Job.connを*pgx.Conn→*sql.Connに変更stdlib.AcquireConn→func (*Conn) Rawで都度*pgx.Connに変換stdlib.ReleaseConn→func (*Conn) CloseLockJob、DoneのメソッドについてxxxContextメソッドを追加NewClient→MustNewClientNewClient2→NewClientPrepareStatementsのcontext対応GetConnectorXXXの追加func (j *Job) ConnにDeprecated:コメントを追加pgx.Conn.AfterConnect→stdlib.OptionAfterConnectまた、関連して以下の修正を行いました
go 1.18→go 1.21docker-compose→docker compose空白を無視すると見やすいかもしれません
https://github.com/kanmu/qg/pull/32/files?w=1