Re: Support of Regex in CalciteAssert

2019-01-02 Thread Michael Mior
+1 to what Julian said. -- Michael Mior mm...@apache.org Le mer. 2 janv. 2019 à 19:57, Julian Hyde a écrit : > I wouldn’t do regex by default. Quite a few characters are regex meta > characters (e.g. “(“ and “.”) and it’s a pain to have to remember to escape > them when copy-pasting output

Re: Support of Regex in CalciteAssert

2019-01-02 Thread Julian Hyde
I wouldn’t do regex by default. Quite a few characters are regex meta characters (e.g. “(“ and “.”) and it’s a pain to have to remember to escape them when copy-pasting output into the test. So, my vote is for > .returns(regexMatch("id=\\p{Graph}+; a=1")) // separate consumer I don’t have an

Support of Regex in CalciteAssert

2019-01-02 Thread Andrei Sereda
Hello, As part of [CALCITE-2755](https://issues.apache.org/jira/browse/CALCITE-2755) ([982](https://github.com/apache/calcite/pull/982)) I needed to perform regex comparison of result (since it is not known in advance). The assertion looks like: ```java assertThat() .query("select id, a from