[
https://issues.apache.org/jira/browse/CASSANDRA-10753?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15029366#comment-15029366
]
Stefania edited comment on CASSANDRA-10753 at 11/27/15 3:38 AM:
----------------------------------------------------------------
Thanks for your review:
bq. In test_timestamp_output since the TZ property is ignored I guess we can
just remove the second equals statement which is the same as the first. What do
you think?
Agreed, removed.
bq. The max_window_size_seconds problem is also affecting 2.2, could you
provide a 2.2 patch? (It's not on jenkins but since we'll still maintain 2.2
for some time, I guess we should fix it).
To fix 2.2 I think we need the entire patch since according to [~aholmber]
[comment|https://issues.apache.org/jira/browse/CASSANDRA-9302?focusedCommentId=15027191&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15027191]
on CASSANDRA-9302 we should be running the current driver. So I backported the
entire patch to 2.2 (via cherry-pick of individual commits) and then upgraded
the bundled driver to the same as 3.0. By the way, do you know which version it
is? I could not even find commit {{3f15725}} on any branch in the driver
repository. Shouldn't we upgrade to a later and more official version instead?
I also started noticing some timeout failures in the cqlshlib tests, on all
branches, when dropping keyspaces. I did not notice them before and I think
it's because I just migrated to a loan laptop where I installed Linux on an
hybrid HDD rather than SSD. It takes a crazy long time to flush all tables
before the final KEYSPACE DROPPED notification is sent to the driver. For 2.2 I
needed at least 20 seconds but for 3.0+ it took up to 35 seconds so I increased
the timeout to 45 seconds in 3.0+. Do you think this is normal?
Here are the patches and CI, but CASSCI is currently down so I cannot start any
jobs:
||2.2||3.0||trunk||
|[patch|https://github.com/stef1927/cassandra/commits/10753-2.2]|[patch|https://github.com/stef1927/cassandra/commits/10753-3.0]|[patch|https://github.com/stef1927/cassandra/commits/10753]|
|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-10753-2.2-dtest/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-10753-3.0-dtest/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-10753-dtest/]|
I verified the cqlshlib tests pass locally on all branches _provided the driver
version installed is the same as the bundled version_. With 3.0rc1 installed,
{{TestCqlshCompletion.test_complete_in_delete}} failed the completion of a
clustering key in the where clause. I did not look further into this, we need
to confirm the correct driver version first.
I'll start the dtests as soon as CASSCI is back.
was (Author: stefania):
Thanks for your review:
bq. In test_timestamp_output since the TZ property is ignored I guess we can
just remove the second equals statement which is the same as the first. What do
you think?
Agreed, removed.
bq. The max_window_size_seconds problem is also affecting 2.2, could you
provide a 2.2 patch? (It's not on jenkins but since we'll still maintain 2.2
for some time, I guess we should fix it).
To fix 2.2 I think we need the entire patch since according to [~aholmber]
[comment|https://issues.apache.org/jira/browse/CASSANDRA-9302?focusedCommentId=15027191&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15027191]
on CASSANDRA-9302 we should be running the current driver. So I backported the
entire patch to 2.2 (via cherry-pick of individual commits) and then upgraded
the bundled driver to the same as 3.0. By the way, do you know which version it
is? I could not even find commit {{3f15725}} on any branch in the driver
repository. Shouldn't we upgrade to a later and more official version instead?
I also started noticing some timeout failures in the cqlshlib tests, on all
branches, when dropping keyspaces. I did not notice them before and I think
it's because I just migrated to a loan laptop where I installed Linux on an
hybrid HDD rather than SSD. It takes a crazy long time to flush all tables
before the final KEYSPACE DROPPED notification is sent to the driver. For 2.2 I
needed at least 20 seconds but for 3.0+ it took up to 35 seconds so I increased
the timeout to 45 seconds in 3.0+. Do you think this is normal?
Here are the patches and CI, but CASSCI is currently down so I cannot start any
jobs:
||2.2||3.0||trunk||
|[patch|https://github.com/stef1927/cassandra/commits/10753-2.2]|[patch|https://github.com/stef1927/cassandra/commits/10753-3.0]|[patch|https://github.com/stef1927/cassandra/commits/10753]|
|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-10753-2.2-dtest/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-10753-3.0-dtest/]|[dtest|http://cassci.datastax.com/view/Dev/view/stef1927/job/stef1927-10753-dtest/]|
I verified the cqlshlib tests locally on all branches. I'll start the dtests as
soon as CASSCI is back.
> Fix completion problems breaking clqshlib tests
> -----------------------------------------------
>
> Key: CASSANDRA-10753
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10753
> Project: Cassandra
> Issue Type: Bug
> Reporter: Stefania
> Assignee: Stefania
> Fix For: 3.0.x
>
>
> The recent changes in the python driver APIs have caused some completion
> problems as indicated by 2 of these failing tests:
> http://cassci.datastax.com/job/trunk_cqlshlib/579/testReport/
> The third failing test, {{test_timestamp_output}}, has been failing for some
> time due to uncertainty on what to do regarding timezone conversion but it
> too can be changed to reflect the fact that we convert the timestamp to UTC.
> Finally, {{max_window_size_seconds}} was recently added to the compaction
> properties and this caused 2 more tests to fail. It cannot be seen on Jenkins
> because of the relative import problem introduced by CASSANDRA-9304.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)