You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Andrey Mashenkov (Jira)" <ji...@apache.org> on 2023/02/08 09:00:00 UTC

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

Andrey Mashenkov created IGNITE-18757:
-----------------------------------------

             Summary: 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


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)