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

undefined ViewSelectPositionStart & ViewSelectPositionEnd #181

Closed
nathan-fiscaletti opened this issue Aug 31, 2020 · 4 comments
Closed

undefined ViewSelectPositionStart & ViewSelectPositionEnd #181

nathan-fiscaletti opened this issue Aug 31, 2020 · 4 comments

Comments

@nathan-fiscaletti
Copy link

I'm getting this on both macOS and Linux running go1.14 (I get it during installation of the package and then when I try to compile a program that uses the package)

# github.com/liquidata-inc/go-mysql-server/sql/parse
../go/src/github.com/liquidata-inc/go-mysql-server/sql/parse/parse.go:792:22: c.ViewSelectPositionStart undefined (type *sqlparser.DDL has no field or method ViewSelectPositionStart)
../go/src/github.com/liquidata-inc/go-mysql-server/sql/parse/parse.go:792:48: c.ViewSelectPositionEnd undefined (type *sqlparser.DDL has no field or method ViewSelectPositionEnd)
@zachmu
Copy link
Member

zachmu commented Aug 31, 2020

H Nathan,

Thanks for notifying us. We're deploying a fix momentarily and will let you know when we have.

@zachmu
Copy link
Member

zachmu commented Aug 31, 2020

This is now fixed on master. I just ran all tests on master and they pass. Please confirm.

I'm a little confused how you ran into this. The breaking change you're talking about was introduced into liquidata-inc/vitess, but liquidata-inc/go-mysql-server didn't update its version of the same until that most recent PR (which also updated the symbols to fix the break). Did you run go get for the @master version of vitess?

@nathan-fiscaletti
Copy link
Author

I discovered this library and went to test it out using the sample code in the read-me when I ran into it.

I only ran the two following go get commands to test it.

$ go get github.com/liquidata-inc/go-mysql-server
$ go get github.com/stretchr/testify/require

All of this happened maybe 10m before actually creating the issue here. So I'm as confused as you are 🤷‍♂️

@zachmu
Copy link
Member

zachmu commented Sep 4, 2020

Well, should be fixed now regardless. Thanks for the report!

@zachmu zachmu closed this as completed Sep 4, 2020
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

No branches or pull requests

2 participants