cube-js/cube

Elasticsearch driver does not work with opendistro

Open

#2,370 opened on Mar 15, 2021

View on GitHub
 (3 comments) (0 reactions) (0 assignees)Rust (19,563 stars) (1,965 forks)batch import
community supportedhelp wanted

Description

When we use opendistro with elastic search driver it fails with below error

Error: "{\n \"error\": {\n \"reason\": \"Invalid SQL query\",\n \"details\": \"Failed to parse query due to offending symbol [:] at: 'SELECT FLOOR((TIMESTAMP_DIFF('seconds', '1970-01-01T00:00:00.000Z':' <--- HERE... More details: Expecting tokens in {'SELECT', '('}\",\n \"type\": \"SyntaxAnalysisException\"\n },\n \"status\": 400\n}"

Expected behavior Expected behavior is that cubejs will work normally without any issue on opendistro

Version: 0.26.53

Contributor guide

Tech stack
elasticsearchtypescriptnodejs
Domain
backenddatabase
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
4
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
3-5 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Elasticsearch knowledgeCube.js basicsSQL parsing
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
30
Research direction
The issue occurs when using opendistro with the Elasticsearch driver, likely due to SQL query parsing differences. Investigate the Elasticsearch driver implementation in cube.js, particularly the SQL generation for timestamp functions. Look at files like 'packages/cubejs elasticsearch driver' and related modules. The error suggests a syntax issue with TIMESTAMP DIFF. Check how the driver constructs date queries and ensure compatibility with opendistro's SQL dialect.