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

Dmitry Konstantinov commented on CASSANDRA-21152:
-------------------------------------------------

Hi [~arvindk12] , thank you for the PR. I am bit confused with the suggested 
change: it looks like it contains both changes disabling of MV support as well 
as changes in the merging logic. Are they both done to fix the same issue or 
the changes in the merging logic about a different issue (if so - do you have a 
test for the correspondent use case)? If both changes are about the same issue 
then why do we need both at the same time? (if disabling of MV support actually 
prevents the changes to use).

> Test failure:  dtest.TestMaterializedViews.test_mv_with_default_ttl_with_flush
> ------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-21152
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-21152
>             Project: Apache Cassandra
>          Issue Type: Bug
>            Reporter: Dmitry Konstantinov
>            Assignee: Dmitry Konstantinov
>            Priority: Normal
>             Fix For: 5.x
>
>         Attachments: breaking_point.png
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> It fails consistently now, example: 
> [https://ci-cassandra.apache.org/job/Cassandra-trunk/2391/testReport/junit/dtest.materialized_views_test/TestMaterializedViews/Tests___dtest_jdk11_1_64___test_mv_with_default_ttl_with_flush/]
> {code:java}
> self = <materialized_views_test.TestMaterializedViews object at 
> 0x7f808eb3b3d0>
>     @since('3.0')
>     def test_mv_with_default_ttl_with_flush(self):
> >       self._test_mv_with_default_ttl(True)
> materialized_views_test.py:1333: 
> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
> _ 
> materialized_views_test.py:1368: in _test_mv_with_default_ttl
>     assert_none(session, "SELECT k,a,b FROM mv2")
> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
> _ 
> session = <cassandra.cluster.Session object at 0x7f808dcc1d60>
> query = 'SELECT k,a,b FROM mv2', cl = None
>     def assert_none(session, query, cl=None):
>         """
>         Assert query returns nothing
>         @param session Session to use
>         @param query Query to run
>         @param cl Optional Consistency Level setting. Default ONE
>     
>         Examples:
>         assert_none(self.session1, "SELECT * FROM test where key=2;")
>         assert_none(cursor, "SELECT * FROM test WHERE k=2", 
> cl=ConsistencyLevel.SERIAL)
>         """
>         simple_query = SimpleStatement(query, consistency_level=cl)
>         res = session.execute(simple_query)
>         list_res = _rows_to_list(res)
> >       assert list_res == [], "Expected nothing from {}, but got 
> > {}".format(query, list_res)
> E       AssertionError: Expected nothing from SELECT k,a,b FROM mv2, but got 
> [[1, 1, None]]
> tools/assertions.py:149: AssertionError
> {code}
> !breaking_point.png|width=700!
> it was broken between 2364 and 2367 Cassandra trunk runs.
> [https://butler.cassandra.apache.org/#/ci/upstream/workflow/Cassandra-trunk/failure/materialized_views_test/TestMaterializedViews/test_mv_with_default_ttl_with_flush]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to