panbingkun
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] ***...
cc @LuciferYang
cc @cloud-fan @yaooqinn @HyukjinKwon
also cc @MaxGekk
After the pr: - When the number of the parameters is `2`, an error message is prompted: - dataset's head has been corrected:
cc @HyukjinKwon @cloud-fan
also cc @MaxGekk
> 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...