Yong Wang

Results 17 comments of Yong Wang

> > the current change is the entry point of kql parsing , if encountered TokenType::ErrorMaxQuerySizeExceeded, it will throw exception, and will not go to other parser > > But...

@antaljanosbenjamin the issues are fixed: ``` SET dialect = 'kusto'; SET max_query_size = 16; Customers | where Education contains 'degree' Syntax error: failed at position 13 ('where'): Customers | where...

Hi @antaljanosbenjamin , there are some test failures ,but I did not find anything related with my commit, can you help to take a look ? Thanks

> I think overall the PR is mergable, I don't see any possible dangerous issues. > > However in the future (not in this PR though) this should be fixed:...

> So the issue is we recently made some changes in master in order to enforce that queries are formatted and parsed properly. That caused a lot of noise in...

> I have rerun the clang-tidy build, because that is required check. After that I think the PR can be merged. thanks

+1 we would like to have this