[
https://issues.apache.org/jira/browse/CALCITE-6908?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18082494#comment-18082494
]
Stamatis Zampetakis commented on CALCITE-6908:
----------------------------------------------
The 33% coverage seems quite low and some classes do not seem to be covered at
all. If we are talking about a production ready module I would expect the tests
to cover at least 70% of the code. Note that the overall coverage in Calcite is
~80% so adding a module with 30% will bring the overall number down.
> Apache Kvrocks adapter
> ----------------------
>
> Key: CALCITE-6908
> URL: https://issues.apache.org/jira/browse/CALCITE-6908
> Project: Calcite
> Issue Type: New Feature
> Reporter: Caican Cai
> Assignee: Caican Cai
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.42.0
>
>
> Apache Kvrocks is a distributed key value NoSQL database that uses RocksDB as
> storage engine and is compatible with Redis protocol and And kvrocks also
> supports querying data through sql , we can adapt it, maybe most of it is
> similar to the redis adapter. But I may want to write a new module and
> refactor and solve some problems of the current redis adapter, such as
> adapting redis sql and compatibility with more redis commands.
> https://github.com/apache/kvrocks
> https://kvrocks.apache.org/docs/supported-commands#search-commands
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
