Re: Found a typo in Catalyst's exception and want to write a test -- help needed

2016-08-17 Thread Reynold Xin
I'd use the new SQLQueryTestSuite. Test cases defined in sql files. On Wed, Aug 17, 2016 at 11:46 PM, Jacek Laskowski wrote: > Hi devs, > > While reviewing the code in Catalyst for doing query parsing I found > that UnresolvedStar has this typo in the exception [1]. > > I do understand that it'

Found a typo in Catalyst's exception and want to write a test -- help needed

2016-08-17 Thread Jacek Laskowski
Hi devs, While reviewing the code in Catalyst for doing query parsing I found that UnresolvedStar has this typo in the exception [1]. I do understand that it's a very trivial issue but I thought I'd write a test for it as part of the change so I could improve my understanding of the low-level bit