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
bug about regex #46
Comments
It is a problem. The current workaround is adding a white space like you said. I'll fix it later. And contribution is welcome😆 |
@jingyugao In your case:
I want to know why your select fields are dynamic. Is there any use case that people can't set their select fields? |
@jingyugao I found that
So can you execute |
|
The marker is not identifiers. It is variable such as 1,"mysql" and so on. `package main import (
) func main() { |
190701 8:30:32 4 Connect root@172.20.0.1 on db0 |
this is the mysql log |
Ok,明天上午之前修复这个问题 |
@jingyugao fixed, see the latest release |
https://github.com/didi/gendry/blob/master/builder/builder.go#L394
this regex is greedy. It should be non-greedy.
for example,
"select {{name}},{{comment}},value from table"
In mysql, this is ok. but gendry will parser it as "name}},{{comment".
I must insert an space between }} and {{ .
The text was updated successfully, but these errors were encountered: