zoomdata/edc-cratedb

edc-cratedb's Issues

Cannot get the EDC crate connector working with Zoomdata

edc-cratedb: master

Zoomdata v2.2.1, v2.3.1, v2.3.10
OS: Ubuntu 16.04 and CentOS 7
crate-jdbc: 1.13.1

The edc-crate connector is started as following:

 java -jar target/connector-server-cratedb-1.1.1-exec.jar

I am able to create the CrateDB data source in Zoomdata. But the exception
occurs when it is used to connect to CrateDB. The edc-crate connector logs
the following message:

WARN 51396 --- [tp1898220577-17] org.eclipse.jetty.http.HttpParser: bad HTTP parsed: 400 for HttpChannelOverHttp@3893657e{r=0,c=false,a=IDLE,uri=null}

and the connection validation fails.

I have tried to build the connector before this commit and it works just perfectly.
It might be that this commit either breaks something or the readme does not
reflect the current state of the project.

Impossibility to provide "Custom SQL" for Crate.io source

While configure Crate.io data source using ZoomData dialog, there is no option to specify "Custom SQL".

Steps to reproduce:

1. Create Connector server (according to video)
2. Create Connection type (according to video)
3. Configure data source in ZoomData
4. On "Tables" tab - there is no option "Custom SQL"

Custom SQL option available for other source (e.g. Impala)

start-server.sh contains the wrong version of the edc crate connector.

start-server.sh != pom.xml

Failed to establish connection due to parsing error

After adding a new connection following the steps at https://github.com/Zoomdata/edc-cratedb the connection validation fails with an Failed to establish connection error.

After creating the connection with a valid JDBC URL and username the following request ist sent from the web UI on "Validate":

http://127.0.0.1/zoomdata/service/connections/validate

Payload:

{"type":"EDC2","subStorageType":"CRATEDB","connectionTypeId":"59bfabe7e4b05a170ecb436b","name":"CrateDb Connection","parameters":{"JDBC_URL":"crate://192.168.43.192:5432/","USER_NAME":"crate"}}

Response:

{"details":"Failed to establish connection","error":"BAD_REQUEST"}

The following output is from the connector server:

2017-09-18 14:06:08.493 WARN 6230 --- [qtp943081537-18] org.eclipse.jetty.http.HttpParser : bad HTTP parsed: 400 for HttpChannelOverHttp@29c1a76f{r=0,c=false,a=IDLE,uri=null}

I've tried with different parameters but can't get it to work.
For me it seems that this is not an error regarding the jdbc connection/properties rather than a connection issue between zoomdata and the connector server?

No refresh/rebuild after change chart settings

Chart is plotted only with default settings (scale, attributes to group etc). In case one change any settings (add filtering option, change grouping attribute, change time scale, update time range in time bar etc) - new chart won't be plotted.

Steps to reproduce:

1. Configure Crate.io data source
2. Add at least one chart for that data source
3. Open that chart
4. After chart is open and rendered try to change scale/add filters/change group att
5. New chart with new settings won't be plotted (only three continuously blinking three dots in left upper corner)

Error on querying uppercase columns

When a datasource is added that contains uppercase "fields" e.g.: CITY - loading a chart fails with the following error:

"ExecuteException(message:io.crate.shade.org.postgresql.util.PSQLException: ERROR: ColumnUnknownException: Column city unknown)"

I guess this is caused due to a missing quotation when querying CrateDB since a direct query to CrateDB without the quotation would fail with the same error.

E.g.:
SELECT CITY FROM address GROUP BY CITY ;
Must be written as:
SELECT "CITY" FROM address GROUP BY "CITY" ;
in order to work.

My current workaround is to use a custom query with an alias like:

SELECT "CITY" as city FROM address GROUP BY city

Importing the test data into Cratedb should be changed

We strongly discourage importing the data as it is done here.

Please use the copy to to persist your test data to a file.
Mount a folder with the file or copy it to a docker container.
Import data from the json file into cratedb using copy from.