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

Stefania commented on CASSANDRA-9303:
-------------------------------------

bq. So just reverting to the previous approach should be fine, .... Regarding 
the printing of the read options, I was thinking of something more concise 
instead of a one-config-per-line which can get too verbose, ...

Done, check reverted and we no longer print each option on a separate line but 
only once per section.

bq. Regarding COPY TO STDOUT should we skip printing info messages since a user 
may want to redirect the output to another script or file?

It's done but I had to move the static methods into {{CopyTask}} so the diff is 
a bit hard, sorry about it.

bq. f I have an {{import.cql}} file containing {{COPY keyspace1.standard1 from 
stdin;}} is the following supposed to work: {{cat input.csv | bin/cqlsh -f 
import.cql?}}

It doesn't work because {{stdin}} is actually set to the file specified with 
the {{-f}} option. Since this is not an issue with COPY but with the way {{-f}} 
is implemented, I would prefer deferring to another ticket if this 
functionality is required.

I've also rebased on the 2.1 branch (since CASSANDRA-9494 will only be on 
trunk) and applied the fix for CASSANDRA-10854 since it requires extra work on 
this branch.

I'm still waiting to hear about which branches we need to apply this patch to; 
plus I would like to squash the commits before up-merging. I would like to 
squash the dtest commits as well, let me know if you still need to review some 
individual commits first.

> Match cassandra-loader options in COPY FROM
> -------------------------------------------
>
>                 Key: CASSANDRA-9303
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9303
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Tools
>            Reporter: Jonathan Ellis
>            Assignee: Stefania
>            Priority: Critical
>             Fix For: 2.1.x
>
>
> https://github.com/brianmhess/cassandra-loader added a bunch of options to 
> handle real world requirements, we should match those.



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

Reply via email to