(calcite) branch main updated: [CALCITE-5613] Assert for number of args for metadata methods at CacheGeneratorUtil

[mbudiu]

This is an automated email from the ASF dual-hosted git repository.

mbudiu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/main by this push:
     new dac001d8ce [CALCITE-5613] Assert for number of args for metadata 
methods at CacheGeneratorUtil
dac001d8ce is described below

commit dac001d8ce7199e84f1148ea7bb57bc2dd83ea91
Author: bpranava <pranava...@gmail.com>
AuthorDate: Fri Aug 30 00:14:18 2024 +0530

    [CALCITE-5613] Assert for number of args for metadata methods at 
CacheGeneratorUtil
---
 .../java/org/apache/calcite/rel/metadata/janino/CacheGeneratorUtil.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/core/src/main/java/org/apache/calcite/rel/metadata/janino/CacheGeneratorUtil.java
 
b/core/src/main/java/org/apache/calcite/rel/metadata/janino/CacheGeneratorUtil.java
index 4d2e3fa13f..f4afe69abb 100644
--- 
a/core/src/main/java/org/apache/calcite/rel/metadata/janino/CacheGeneratorUtil.java
+++ 
b/core/src/main/java/org/apache/calcite/rel/metadata/janino/CacheGeneratorUtil.java
@@ -209,6 +209,8 @@ class CacheGeneratorUtil {
 
       /** Returns e.g. ", ignoreNulls". */
       private StringBuilder safeArgList(StringBuilder buff, Method method) {
+        assert method.getParameterCount() >= 2
+            : "At least 2 metadata method parameters are required";
         // We ignore the first 2 arguments since they are included other ways.
         for (Ord<Class<?>> t : Ord.zip(method.getParameterTypes())
             .subList(2, method.getParameterCount())) {