You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "xiong duan (Jira)" <ji...@apache.org> on 2023/08/03 00:59:00 UTC
[jira] [Updated] (CALCITE-5875) Remove unnecessary NULL checks in Redis adapter
[ https://issues.apache.org/jira/browse/CALCITE-5875?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
xiong duan updated CALCITE-5875:
--------------------------------
Summary: Remove unnecessary NULL checks in Redis adapter (was: Remove unnecessary NULL checks in Redis adapter")
> Remove unnecessary NULL checks in Redis adapter
> -----------------------------------------------
>
> Key: CALCITE-5875
> URL: https://issues.apache.org/jira/browse/CALCITE-5875
> Project: Calcite
> Issue Type: Improvement
> Affects Versions: 1.34.0
> Reporter: Ran Tao
> Assignee: Ran Tao
> Priority: Minor
> Labels: pull-request-available
> Fix For: 1.36.0
>
>
> It's minor fix to remove some unnecessary null checks for redis-adapter.
> e.g.
> {code:java}
> if (REDIS_CONTAINER != null && REDIS_CONTAINER.isRunning()) {
> REDIS_CONTAINER.stop();
> } {code}
> however REDIS_CONTAINER is static variable must not be null unless cause exception.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)