You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Sergey Nuyanzin (Jira)" <ji...@apache.org> on 2023/05/07 18:52:00 UTC
[jira] [Commented] (CALCITE-5611) Show SQL query for failed tests for SqlOperatorTest
[ https://issues.apache.org/jira/browse/CALCITE-5611?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17720350#comment-17720350 ]
Sergey Nuyanzin commented on CALCITE-5611:
------------------------------------------
Thanks for having a look,
yes in some cases it would make sense to use {{Supplier<String>}} rather than {{String}}
I applied it
> Show SQL query for failed tests for SqlOperatorTest
> ---------------------------------------------------
>
> Key: CALCITE-5611
> URL: https://issues.apache.org/jira/browse/CALCITE-5611
> Project: Calcite
> Issue Type: Improvement
> Components: tests
> Affects Versions: 1.34.0
> Reporter: Sergey Nuyanzin
> Priority: Major
> Labels: pull-request-available
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> The issue is that there are lots of cases under each test in {{SqlOperatorTest}} and it is not clear which of them is failing.
> The idea is to print sql query from the test if it's failed besides type/result comparison
--
This message was sent by Atlassian Jira
(v8.20.10#820010)