[
https://issues.apache.org/jira/browse/CALCITE-7388?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18053232#comment-18053232
]
terran2010 commented on CALCITE-7388:
-------------------------------------
[~mbudiu] Thank you for your reply. This file is used as a bridge for
converting Redis and Calcite data. I noticed that there is currently no
documentation on the use of this part on the official website of Cacltie. We
can consider providing an explanation on this part to make people more aware of
the valid values of this configuration file. I think the verification of the
valid values of the file should still be completed in Cacltie's Redis adapter
> Redis Adapter operand config should not support empty string
> ------------------------------------------------------------
>
> Key: CALCITE-7388
> URL: https://issues.apache.org/jira/browse/CALCITE-7388
> Project: Calcite
> Issue Type: Bug
> Affects Versions: 1.41.0
> Reporter: terran2010
> Priority: Major
> Labels: pull-request-available
>
> The normal config
> {code:java}
> "operand": {
> "dataFormat": "csv",
> "keyDelimiter": ":",
> "fields": [
> {
> "name": "DEPTNO",
> "type": "varchar",
> "mapping": 0
> },
> {
> "name": "NAME",
> "type": "varchar",
> "mapping": 1
> }
> ]
> } {code}
> but this config
> {code:java}
> "operand": {
> "dataFormat": "",
> "keyDelimiter": ":",
> "fields": []
> } {code}
> It was successfully loaded into the Redis data model, resulting in errors
> when executing SQL. Normally, illegal key parameters should be disabled when
> building metadata. We can consider optimizing it
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
