Commit 0341c89
committed
Support serializing & deserializing UDTs when pushing down scripts (opensearch-project#4245)
* Support serializing & deserializing UDTs
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Update explain ITs
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Push down UDT types as string types for comparison operators
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Separate test cases and add an ignored IT
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Correct the handling of UDT in CalciteScriptEngine by substituting calcite's type factory with OpenSearchTypeFactory
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Fix deserialization for IP
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Remove testExplainPushDownScriptsContainingUDT in v2
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Enable testLimitAfterAggregation in CalcitePPLAggregationIT
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Unit test serialize map and array types
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Fix deeper level deserialization of UDTs
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Add a yaml test for issue 4322
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Add a test case for issue 4340
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
* Remove redundant classes
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
---------
Signed-off-by: Yuanchun Shen <yuanchu@amazon.com>
(cherry picked from commit 69a718b)1 parent 2c41580 commit 0341c89
16 files changed
Lines changed: 1521 additions & 14 deletions
File tree
- integ-test/src
- test
- java/org/opensearch/sql/calcite/remote
- resources/expectedOutput
- calcite_no_pushdown
- calcite
- yamlRestTest/resources/rest-api-spec/test/issues
- opensearch/src
- main/java/org/opensearch/sql/opensearch/storage
- script
- serde
- test/java/org/opensearch/sql/opensearch/storage/serde
Lines changed: 0 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
204 | 204 | | |
205 | 205 | | |
206 | 206 | | |
207 | | - | |
208 | | - | |
209 | 207 | | |
210 | 208 | | |
211 | 209 | | |
| |||
Lines changed: 28 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| 13 | + | |
13 | 14 | | |
14 | 15 | | |
15 | 16 | | |
| |||
1082 | 1083 | | |
1083 | 1084 | | |
1084 | 1085 | | |
| 1086 | + | |
| 1087 | + | |
| 1088 | + | |
| 1089 | + | |
| 1090 | + | |
| 1091 | + | |
| 1092 | + | |
| 1093 | + | |
| 1094 | + | |
| 1095 | + | |
| 1096 | + | |
| 1097 | + | |
| 1098 | + | |
| 1099 | + | |
| 1100 | + | |
| 1101 | + | |
| 1102 | + | |
| 1103 | + | |
| 1104 | + | |
| 1105 | + | |
| 1106 | + | |
| 1107 | + | |
| 1108 | + | |
| 1109 | + | |
| 1110 | + | |
| 1111 | + | |
| 1112 | + | |
1085 | 1113 | | |
1086 | 1114 | | |
1087 | 1115 | | |
| |||
Lines changed: 39 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1220 | 1220 | | |
1221 | 1221 | | |
1222 | 1222 | | |
| 1223 | + | |
| 1224 | + | |
| 1225 | + | |
| 1226 | + | |
| 1227 | + | |
| 1228 | + | |
| 1229 | + | |
| 1230 | + | |
| 1231 | + | |
| 1232 | + | |
| 1233 | + | |
| 1234 | + | |
| 1235 | + | |
| 1236 | + | |
| 1237 | + | |
| 1238 | + | |
| 1239 | + | |
| 1240 | + | |
| 1241 | + | |
| 1242 | + | |
| 1243 | + | |
| 1244 | + | |
| 1245 | + | |
| 1246 | + | |
| 1247 | + | |
| 1248 | + | |
| 1249 | + | |
| 1250 | + | |
| 1251 | + | |
| 1252 | + | |
| 1253 | + | |
| 1254 | + | |
| 1255 | + | |
| 1256 | + | |
| 1257 | + | |
| 1258 | + | |
| 1259 | + | |
| 1260 | + | |
| 1261 | + | |
1223 | 1262 | | |
1224 | 1263 | | |
1225 | 1264 | | |
| |||
Lines changed: 2 additions & 2 deletions
Large diffs are not rendered by default.
Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
Lines changed: 6 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
Lines changed: 6 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
0 commit comments