You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Evgeny Stanilovsky (Jira)" <ji...@apache.org> on 2025/10/20 06:26:02 UTC

[jira] [Updated] (IGNITE-18757) Sql. Add arch unit tests for classes that inherited from Calcite classes.

     [ https://issues.apache.org/jira/browse/IGNITE-18757?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Evgeny Stanilovsky updated IGNITE-18757:
----------------------------------------
    Fix Version/s: 3.2
                       (was: 3.1)

> Sql. Add arch unit tests for classes that inherited from Calcite classes.
> -------------------------------------------------------------------------
>
>                 Key: IGNITE-18757
>                 URL: https://issues.apache.org/jira/browse/IGNITE-18757
>             Project: Ignite
>          Issue Type: Improvement
>          Components: sql
>            Reporter: Andrey Mashenkov
>            Priority: Major
>              Labels: ignite-3, tech-debt
>             Fix For: 3.2
>
>
> Motivation: as of now, it is easy to miss/forget to override methods, when inherit from Calcite classes.
> E.g. from time to time we forget to override methods for deriving/passing through traits or passing shuttle.
> We can use ArchUnit to make additional checks that all required methods are overridden.
> As an example, see static logger checks in ClientArchTest.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)