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

Vladimir Sitnikov commented on CALCITE-469:
-------------------------------------------

I see more problems with CSV readme:
1) It points to outdated {{git://github.com/julianhyde/optiq-csv.git}}
2) Current [https://github.com/apache/incubator-calcite] depends on the 
SNAPSHOT version, thus to test the CVS one needs to build full Calcite.

Taking 1&2 into account I think the proper way to test CSV adapter is as 
follows:
1) Download Calcite from the github mirror
2) mvn install {{-DskipTests -Dcheckstyle.skip=true}}
3) {{cd examples/csv}}, etc.

And I would like to ensure all queries are copy-pasteable. E.g. "sqlline>" does 
not appear in the middle of the query.

Any objections/comments?

> README.md instruction for compile in example for csv doesn't work
> -----------------------------------------------------------------
>
>                 Key: CALCITE-469
>                 URL: https://issues.apache.org/jira/browse/CALCITE-469
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Larry McCay
>            Assignee: Julian Hyde
>         Attachments: OPTIQ-469.patch
>
>
> mvn compile doesn't put the required files in the expected place for the 
> tutorial. Change to 'mvn clean install'.
> I will provide the oneliner patch. :)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to