[Bug] repeat function has hard limit 10000

[zhangxffff]

Component Selection

• Core Engine (Expression eval, Memory, Vector)
• Connectors / File Formats (Hive, Parquet, etc.)
• API / Bindings (Python, etc.)
• Build
• Other

Describe the Bug

org.apache.gluten.exception.GlutenException: Exception: BoltUserError
Error Source: USER
Error Code: INVALID_ARGUMENT
Reason: (10437 vs. 10000) Count argument of repeat function must be less than or equal to 10000
Retriable: False
Expression: count <= kMaxResultEntries
Context: array_repeat(get(n21_4, 0:INTEGER), coalesce(cast((get(n21_4, 1:INTEGER)) as INTEGER), 1:INTEGER))
Additional Context: Top-level Expression: concat_ws(,:VARCHAR, array_repeat(get(n21_4, 0:INTEGER), coalesce(cast((get(n21_4, 1:INTEGER)) as INTEGER), 1:INTEGER)))
Function: checkCount
File: /opt/tiger/compile_path/src/code.byted.org/olap/gluten/ep/bolt/bolt/functions/lib/Repeat.cpp
Line: 80

Reproduction Steps

select array_repeat(1, 20000);

Bolt Version / Commit ID

7f4e4f5

System Configuration

- **OS**: (e.g. Ubuntu 22.04, CentOS 7)
- **Compiler**: (e.g. GCC 11, Clang 14)
- **Build Type**: (Debug / Release / RelWithDebInfo)
- **CPU Arch**: (e.g. x86_64 AVX2, ARM64)
- **Framework**: (e.g. Spark 3.3, PrestoDB)

Logs / Stack Trace

Expected Behavior

No response

Additional context

No response

[Vansh0Sharma]

hi @zhangxffff I have submitted a Pull Request (#655) to fix this issue by increasing the kMaxResultEntries limit to 100,000. Please take a look when you have some time!

[zhangxffff]

hi @zhangxffff I have submitted a Pull Request (#655) to fix this issue by increasing the kMaxResultEntries limit to 100,000. Please take a look when you have some time!

The Pull Request seems to have been closed. By the way, we should handle Spark and Presto separately — Presto has a 10,000 limit, but Spark has no such limit.
https://github.com/prestodb/presto/blob/7d161786df59329cee080e9d6326d136f8b81c55/presto-main/src/main/java/com/facebook/presto/operator/scalar/RepeatFunction.java#L146