I haven’t read the full PR. (Sorry, finite time due to $dayjob.) If the PR only removes output from the screen or log during tests, I’m fine.
> On Aug 29, 2018, at 11:55 AM, Vladimir Sitnikov <[email protected]> > wrote: > > Julian>I think we should just solve problem 1 for now. > > Does that mean you agree to merge the PR? > > Julian>For 2, if we continue to write "No match found for function > signature ABCDE(<NUMERIC>, <ANY>, <CHARACTER>)” to the log it’s not perfect > but it’s better than nothing. > > Why do you think this is the only logged message? > > Does that exception die in Calcite sources? I hope it does not. > Does it propagate to the client API? > Vladimir
