Jackie-Jiang commented on pull request #6719: URL: https://github.com/apache/incubator-pinot/pull/6719#issuecomment-825321407
> This is certainly good work. Both BOOLEAN and TIMESTAMP would be very useful. I haven't looked at all the files yet, but will look again later. Not sure if I missed them, but would like to see more SQL level test cases for these two data types. @amrishlal Added `BooleanQueriesTest` and `TimestampQueriesTest` to test the query flow for various types of queries -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
