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

fix: import for cockroach db #1399

Merged
merged 2 commits into from
Mar 14, 2023
Merged

fix: import for cockroach db #1399

merged 2 commits into from
Mar 14, 2023

Conversation

markphelps
Copy link
Collaborator

When working on #1396 I saw that if you tried to run import using cockroachDB you would get a nil pointer error. This is because we never added cockroachDB to the switch statement for import/export commands.. oops

@markphelps markphelps requested a review from a team as a code owner March 14, 2023 16:05
Copy link
Contributor

@GeorgeMac GeorgeMac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops

@markphelps markphelps enabled auto-merge (squash) March 14, 2023 16:08
@markphelps markphelps merged commit 1273bed into main Mar 14, 2023
@markphelps markphelps deleted the fix-cockroach-import branch March 14, 2023 16:09
markphelps added a commit that referenced this pull request Mar 15, 2023
* main: (36 commits)
  feat(sdk/go): thread client token on RPC calls when provided (#1389)
  docs: update .all-contributorsrc [skip ci]
  docs: update README.md [skip ci]
  chore(deps): bump google.golang.org/protobuf in /_tools (#1401)
  chore(deps): bump google.golang.org/protobuf from 1.29.0 to 1.29.1 (#1400)
  fix: return better error messages for grpc-gateway errors (#1397)
  fix: import for cockroach db (#1399)
  chore(deps): bump golang.org/x/tools from 0.6.0 to 0.7.0 in /_tools (#1394)
  chore: update readme w better usecase verbiage (#1395)
  fix(sdk/go): documentation and example corrections
  chore(deps): bump google.golang.org/protobuf from 1.28.1 to 1.29.0 (#1393)
  chore(deps): bump google.golang.org/protobuf in /_tools (#1391)
  chore(sdk/go): fix README markdown style
  fix(sdk/go): drop support for single field message mangling
  chore(sdk/go): add README
  chore(gowork): go edit -fmt
  feat(sdk/go): add doc.go
  feat(sdk/go): add static client token provider
  refactor(sdk/go): move grpc transport into its own subpackage
  chore(proto): regenerate RPC using protoc-gen-go-grpc v1.3.0
  ...
markphelps added a commit that referenced this pull request Mar 15, 2023
* namespaces: (31 commits)
  Release/1.19 (#1403)
  feat(sdk/go): thread client token on RPC calls when provided (#1389)
  docs: update .all-contributorsrc [skip ci]
  docs: update README.md [skip ci]
  chore(deps): bump google.golang.org/protobuf in /_tools (#1401)
  chore(deps): bump google.golang.org/protobuf from 1.29.0 to 1.29.1 (#1400)
  fix: return better error messages for grpc-gateway errors (#1397)
  fix: import for cockroach db (#1399)
  chore(deps): bump golang.org/x/tools from 0.6.0 to 0.7.0 in /_tools (#1394)
  chore: update readme w better usecase verbiage (#1395)
  fix(sdk/go): documentation and example corrections
  chore(deps): bump google.golang.org/protobuf from 1.28.1 to 1.29.0 (#1393)
  chore(deps): bump google.golang.org/protobuf in /_tools (#1391)
  chore(sdk/go): fix README markdown style
  fix(sdk/go): drop support for single field message mangling
  chore(sdk/go): add README
  chore(gowork): go edit -fmt
  feat(sdk/go): add doc.go
  feat(sdk/go): add static client token provider
  refactor(sdk/go): move grpc transport into its own subpackage
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants