You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Zhen Chen (Jira)" <ji...@apache.org> on 2025/12/14 14:31:00 UTC

[jira] [Commented] (CALCITE-7333) The CalciteResource.properties file needs to contain the corresponding properties as in CalciteResource.java

    [ https://issues.apache.org/jira/browse/CALCITE-7333?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18044957#comment-18044957 ] 

Zhen Chen commented on CALCITE-7333:
------------------------------------

We already have this test.
{code:java}
@Test void testResources() {
  Resources.validate(Static.RESOURCE);
  checkResourceMethodNames(Static.RESOURCE);
} {code}

> The CalciteResource.properties file needs to contain the corresponding properties as in CalciteResource.java
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: CALCITE-7333
>                 URL: https://issues.apache.org/jira/browse/CALCITE-7333
>             Project: Calcite
>          Issue Type: Test
>          Components: core
>    Affects Versions: 1.41.0
>            Reporter: Zhen Chen
>            Assignee: Zhen Chen
>            Priority: Minor
>
> If I add the following code to CaliteResource.java:
> {code:java}
> @BaseMessage("For test")
> ExInst<SqlValidatorException> forTest(); {code}
> but don't add the corresponding item to CaliteResource.properties, the compilation test will work. We need to add a test case to ensure that the item in CaliteResource.java has a corresponding item in CaliteResource.properties.



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