Vitess sqlparser panic

[stephenmw]

The following code panics on iteration i == 33
http://play.golang.org/p/xMDjv9I95E

I ran it as a test file in vt/sqlparser.

[stephenmw]

I think this is a yacc bug so I also filed a bug at https://code.google.com/p/go/issues/detail?id=7184

[sougou]

You should get an award for making something as solid as yacc crash :).

On Wed, Jan 22, 2014 at 3:25 PM, Stephen Weinberg
notifications@github.comwrote:

I think this is a yacc bug so I also filed a bug at
https://code.google.com/p/go/issues/detail?id=7184

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/33#issuecomment-33080970
.

[stephenmw]

Heh, thanks. That is what I get for trying crazy queries..

Are you guys just going to rely on the Go team to fix yacc? Have you verified it is yacc's fault?

-- Stephen

[sougou]

We'll wait for the yacc bug outcome. If you indefinitely nest 'select * from(', something will have to eventually go wrong :). If the bug becomes a real concern, we'll just limit nesting to a reasonable number.

[stephenmw]

Also, in case it is of interest to you, mysql caps nesting in select to 63. 64 gives ERROR 1473 (HY000) at line 1: Too high level of nesting for select

I was bored today.

[sougou]

I'll probably have to do something along these lines, like Rob suggested.