[CALCITE-5875] Remove unnecessary NULL checks in Redis adapter

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.34.0
• Fix Version/s: 1.36.0
• Labels: pull-request-available
• Assignee: Ran Tao
• Reporter: Ran Tao
• Created: 27/Jul/23 03:05
• Updated: 10/Nov/23 11:34
• Resolved: 03/Aug/23 00:59

Description

It's minor fix to remove some unnecessary null checks for redis-adapter.
e.g.

if (REDIS_CONTAINER != null && REDIS_CONTAINER.isRunning()) {
  REDIS_CONTAINER.stop();
} 

however REDIS_CONTAINER is static variable must not be null unless cause exception.

Issue Links

• links to: GitHub Pull Request #3339