[FLINK-26474] Fail to call hive udf when secondary parameter required constant but pass value like "-1"

• Type: Sub-task
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 1.16.0
• Component/s: Connectors / Hive
• Labels: pull-request-available
• Assignee: luoyuxia
• Reporter: luoyuxia
• Created: 04/Mar/22 03:29
• Updated: 21/Oct/22 03:57
• Resolved: 31/Aug/22 05:39

Description

The exception can be reproduced using following code in HiveDialectITCase

 HiveModule hiveModule = new HiveModule(hiveCatalog.getHiveVersion());
 CoreModule coreModule = CoreModule.INSTANCE;
 for (String loaded : tableEnv.listModules()) {
   tableEnv.unloadModule(loaded);
}
tableEnv.loadModule("hive", hiveModule);
tableEnv.loadModule("core", coreModule);
List<Row> results = CollectionUtil.iteratorToList(tableEnv.executeSql("select bround(55.0, -1)").collect());

It will case such exception "BROUND second argument only takes constant", the reson is the parameter "-1" will be considered as function call instead of contant, we should fold it in HiveParserTypeCheckProcFactory#getXpathOrFuncExprNodeDesc.

Issue Links

• fixes: FLINK-29703 Fail to call unix_timestamp in runtime in Hive dialect
• links to: GitHub Pull Request #18975