Commit 09fbad5
Use scalarFunctionExprToProtoWithReturnType for levenshtein
Override convert() to use scalarFunctionExprToProtoWithReturnType
with IntegerType, so the native planner skips the DataFusion
registry lookup and does not conflict with DataFusion's built-in
2-arg levenshtein function when 3 args (threshold) are passed.
Also fix Spotless: remove unnecessary string interpolation prefix.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>1 parent 291da46 commit 09fbad5
2 files changed
Lines changed: 16 additions & 3 deletions
File tree
- spark/src
- main/scala/org/apache/comet/serde
- test/scala/org/apache/comet
Lines changed: 15 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
| 25 | + | |
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
| |||
84 | 84 | | |
85 | 85 | | |
86 | 86 | | |
87 | | - | |
| 87 | + | |
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
| |||
96 | 96 | | |
97 | 97 | | |
98 | 98 | | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
99 | 112 | | |
100 | 113 | | |
101 | 114 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
493 | 493 | | |
494 | 494 | | |
495 | 495 | | |
496 | | - | |
| 496 | + | |
497 | 497 | | |
498 | 498 | | |
499 | 499 | | |
| |||
0 commit comments