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)