Skip to content

Commit

Permalink
Accept routing hints in comments (pg-sharding#317)
Browse files Browse the repository at this point in the history
* Accept routing hints in comments
  • Loading branch information
reshke authored and CBists committed Nov 13, 2023
1 parent f2b1424 commit 3847399
Show file tree
Hide file tree
Showing 10 changed files with 122 additions and 90 deletions.
14 changes: 9 additions & 5 deletions pkg/mock/conn/mock_instance.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 10 additions & 6 deletions pkg/mock/conn/raw_mock.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

36 changes: 20 additions & 16 deletions pkg/mock/pgx/mock_pgx_tx.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

12 changes: 8 additions & 4 deletions pkg/mock/pgx/mock_pgxconn_iface.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 10 additions & 6 deletions pkg/mock/shard/mock_shard.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions router/client/client_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import (
"github.com/golang/mock/gomock"
"github.com/jackc/pgx/v5/pgproto3"
"github.com/pg-sharding/spqr/router/client"
"go.uber.org/mock/gomock"

"github.com/golang/mock/gomock"
"github.com/pg-sharding/spqr/pkg/conn"
Expand Down
1 change: 0 additions & 1 deletion router/frontend.go
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ func deparseRouteHint(rst relay.RelayStateMgr, params map[string]string, dataspa
}

meta := qrouter.NewRoutingMetadataContext(krs, rls, dataspace, nil)

ds, err := rst.QueryRouter().DeparseKeyWithRangesInternal(context.TODO(), val, meta)
if err != nil {
return nil, err
Expand Down
Loading

0 comments on commit 3847399

Please sign in to comment.