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

sql: Scan error on column index 2, name "is_primary" #24

Closed
fillon opened this issue Sep 20, 2022 · 4 comments · Fixed by #25
Closed

sql: Scan error on column index 2, name "is_primary" #24

fillon opened this issue Sep 20, 2022 · 4 comments · Fixed by #25

Comments

@fillon
Copy link

fillon commented Sep 20, 2022

Hi,

Just installed mermerd, look like a nice tool.

I tried to run it against a mysql database and get the following error

  • mermerd 0.4.0
  • macos amd
INFO[0006] Got tables                                    count=38
[=>                  ]Getting columns and constraintsERRO[0008] Getting columns failed | sql: Scan error on column index 2, name "is_primary": sql/driver: couldn't convert 2 into type bool
ERRO[0008] sql: Scan error on column index 2, name "is_primary": sql/driver: couldn't convert 2 into type bool


X Something went wrong!

Any idea ? :-)

@KarnerTh
Copy link
Owner

KarnerTh commented Sep 20, 2022

Thanks for pointing out 👍

Yeah I have a good guess what could be wrong, I will look into this

@ashutosrath
Copy link
Contributor

ashutosrath commented Sep 26, 2022

I also got a similar issue.

INFO[0000] Got tables                                    count=13
? Choose tables: clouds, clusters
[=>                  ]Getting columns and constraintsERRO[0009] Getting columns failed | sql: Scan error on column index 3, name "is_foreign": sql/driver: couldn't convert 3 into type bool
ERRO[0009] sql: Scan error on column index 3, name "is_foreign": sql/driver: couldn't convert 3 into type bool

@KarnerTh - Please review and merge the PR #25. This fix worked for me.

@KarnerTh
Copy link
Owner

@ashutosrath thanks for the PR 👍
I will release a new version in the near future

@KarnerTh
Copy link
Owner

The fix is now available in Version 0.4.1

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 a pull request may close this issue.

3 participants