[
https://issues.apache.org/jira/browse/CASSANDRA-11701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15408997#comment-15408997
]
Stefania edited comment on CASSANDRA-11701 at 8/8/16 2:10 AM:
--
I rewrote the patch to avoid using a thread lock, since it cannot be pickl-ed
on Windows. I extracted the pipe out of the channels, so we only transfer pipes
and not channels to the child processes, which then create the channels after
forking. To avoid redundant threads, which are only required for sending, I've
separated receiving from sending channels. Only sending channels create a
feeding thread, right in the constructor so the lock is no longer required.
I'm not sure if we want this fix in 2.1 or not, it is not a critical bug but it
is a regression compared to the old cqlsh COPY functionality. It is a rare
failure but it can occur if the main thread of a child process needs to send an
error when the receiving thread is already sending results.
||2.1||2.2||3.0||3.9||trunk||
|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-2.1]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-2.2]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-3.0]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-3.9]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh]|
|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-2.1-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-2.2-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-3.0-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-3.9-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-cqlsh-tests/]|
The windows test results will be available
[here|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-dtest-multiplex-win32/],
build -#7 and #8- #8 and #9.
was (Author: stefania):
I rewrote the patch to avoid using a thread lock, since it cannot be pickl-ed
on Windows. I extracted the pipe out of the channels, so we only transfer pipes
and not channels to the child processes, which then create the channels after
forking. To avoid redundant threads, which are only required for sending, I've
separated receiving from sending channels. Only sending channels create a
feeding thread, right in the constructor so the lock is no longer required.
I'm not sure if we want this fix in 2.1 or not, it is not a critical bug but it
is a regression compared to the old cqlsh COPY functionality. It is a rare
failure but it can occur if the main thread of a child process needs to send an
error when the receiving thread is already sending results.
||2.1||2.2||3.0||3.9||trunk||
|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-2.1]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-2.2]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-3.0]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh-3.9]|[patch|https://github.com/stef1927/cassandra/commits/11701-cqlsh]|
|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-2.1-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-2.2-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-3.0-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-3.9-cqlsh-tests/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-11701-cqlsh-cqlsh-tests/]|
The windows test results will be available
[here|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-dtest-multiplex-win32/],
build #7 and #8.
> [windows] dtest failure in
> cqlsh_tests.cqlsh_copy_tests.CqlshCopyTest.test_reading_with_skip_and_max_rows
> -
>
> Key: CASSANDRA-11701
> URL: https://issues.apache.org/jira/browse/CASSANDRA-11701
> Project: Cassandra
> Issue Type: Test
>Reporter: Russ Hatch
>Assignee: Stefania
> Labels: dtest, windows
>
> looks to be an assertion problem, so could be test or cassandra related:
> e.g.:
> {noformat}
> 1 != 331
> {noformat}
> http://cassci.datastax.com/job/trunk_dtest_win32/404/testReport/cqlsh_tests.cqlsh_copy_tests/CqlshCopyTest/test_reading_with_skip_and_max_rows
> Failed on CassCI build trunk_dtest_win32 #404
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
