[ 
https://issues.apache.org/jira/browse/CALCITE-1730?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15951641#comment-15951641
 ] 

Julian Hyde commented on CALCITE-1730:
--------------------------------------

OK, the test runs successfully with CALCITE-1734. It is slow 
({{DruidAdapterIT#testFilterTimestamp}} alone took 4m) but it finishes. I was 
faster if I increased the VM's memory from 3G to 6G (see 
https://github.com/julianhyde/calcite-test-dataset/commit/c07da47cfbf724f7c9e3e0b997cc87ced8949831)
 but I think we should keep the base VM at 3G.

Two changes, please:
* Update Calcite's doc (history.md and druid_adapter.md), stating which 
version(s) are supported
* In {{install.sh}}, convert zookeeper version into a variable as you did for 
druid version

When you make those changes I'll +1 your 
https://github.com/vlsi/calcite-test-dataset/pull/19 and will also rebase 
https://github.com/vlsi/calcite-test-dataset/pull/18 and start testing that.

> Upgrade Druid to 0.9.2
> ----------------------
>
>                 Key: CALCITE-1730
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1730
>             Project: Calcite
>          Issue Type: Bug
>          Components: druid
>            Reporter: Julian Hyde
>            Assignee: Nishant Bangarwa
>
> Upgrade Druid to 0.9.2. Since we speak REST to Druid there are no maven 
> dependencies to update, but we do need to make calcite-test-dataset work 
> reliably with a 0.9.2 instance and there are a couple of places in the 
> doc/site IIRC.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to