> I have tried working on Jeff Zucker's SQL::Parser module to fully
> support the IN clause.
Thanks James, what would help most is some sample queries that didn't work before and do work after.
I have done some light testing on this
Unfortunately, the testing bit is the most labor intensive part and the part I need most help on. I can come up with lots of examples that work, but finding the ones that don't isn't always simple. Also making a fix is not as hard as making sure the fix doesn't break other things (e.g., did you test with NOT IN? or where the IN is between other parenthetical clauses?).
Next I'm going to try to get all 3 way joins working.
Not sure exactly what you mean here. It seems to me that the most important thing to tackle is self joins. Also, are you looking at fixes for For SQL::Parser only, or for SQL::Statement as well?
870c870
< if ($str =~ /^(.+?) IN (\(.+)$/i ) {
What version is this a patch against? -- Jeff
