panbingkun

Results 183 comments of panbingkun

> Are the failures relevant? Yes, let me continue investigating it.

> Could you rebase to the `master` branch once more? It seems that we passed the CIs except the known flaky tests, @panbingkun . > > ``` > [info] ***...

After the pr: - When the number of the parameters is `2`, an error message is prompted: - dataset's head has been corrected:

> Do we have a strong reason to make this breaking change? How bad is it when calling `sentences` with two parameters? Mainly because the inconsistent use of function `sentences`...

> We **shall not break** existing SQL user's code only because it is inconsistent to Scala/Python APIs. As I said let's document the existing behaviour of SQL func first of...