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

*: move github.com/pingcap/tidb/parser here #1

Merged
merged 5 commits into from
Oct 24, 2018

Conversation

tiancaiamao
Copy link
Collaborator

Here is the dependency graph:
parser

TiDB need to update its "import github.com/pingcap/tidb/parser" to "import github.com/pingcap/parser"

@tiancaiamao
Copy link
Collaborator Author

@shenli

@kennytm
Copy link
Contributor

kennytm commented Oct 23, 2018

Question about package management... As parser.go isn't checked in, do the dependencies still need to manually make parser and check it into their vendor/*?

@tiancaiamao
Copy link
Collaborator Author

parser.go will be included in this repository, although I didn't do it in this commit @kennytm

@tiancaiamao tiancaiamao merged commit 53ac409 into pingcap:master Oct 24, 2018
@tiancaiamao tiancaiamao deleted the first-commit branch October 24, 2018 08:20
ngaut pushed a commit that referenced this pull request Jan 6, 2019
spongedu pushed a commit to spongedu/parser that referenced this pull request Oct 17, 2019
windtalker pushed a commit to windtalker/parser that referenced this pull request May 6, 2020
tiancaiamao added a commit to tiancaiamao/parser that referenced this pull request Apr 27, 2021
tiancaiamao pushed a commit to tiancaiamao/parser that referenced this pull request Apr 27, 2021
tiancaiamao pushed a commit to tiancaiamao/parser that referenced this pull request Jun 10, 2021
spongedu pushed a commit to spongedu/parser that referenced this pull request Sep 17, 2021
parser, ast: make parser happy for PARTITION BY [LINEAR] KEY ALGORITH…
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