[ https://issues.apache.org/jira/browse/CASSANDRA-9369?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14550496#comment-14550496 ]
Ariel Weisberg commented on CASSANDRA-9369: ------------------------------------------- If we were caught up and all tests were passing then yes disabling failing tests would be a no-no. Right now when we are far behind and trying to catch up it makes sense to disable tests we know aren't passing. See CASSANDRA-8997 and CASSANDRA-9002 for why (and feel free to comment and disagree). Discussing this might be good retrospective fodder. I want to bring up the fact that it's been a few months and utests and dtests still aren't passing, and people are still doing feature work. What's the point of doing feature work for a system that you can't ship anyways because the tests don't pass? I wouldn't be surprised if the tests don't pass by the time we want to ship 2.2. By branching 2.2 we increased test load an complexity because now we have two places to merge to and two sets of tests to monitor. I don't agree with the thinking that addressing test issues delays feature work. It's already delayed you are just in denial. > HSHA dtest for closing connections is failing on trunk > ------------------------------------------------------ > > Key: CASSANDRA-9369 > URL: https://issues.apache.org/jira/browse/CASSANDRA-9369 > Project: Cassandra > Issue Type: Test > Components: Tests > Reporter: Tyler Hobbs > Assignee: Ariel Weisberg > Fix For: 2.2.x > > > The {{thrift_hsha_test.ThriftHSHATest.test_closing_connections}} dtest is > failing against trunk (but not against 2.1). Here's an example failure: > {noformat} > Error Message > There are non-closed connections: COMMAND PID USER FD TYPE DEVICE > SIZE/OFF NODE NAME > java 28202 automaton 7u IPv4 60767 0t0 TCP > localhost:58436->localhost:9160 (CLOSE_WAIT) > -------------------- >> begin captured logging << -------------------- > dtest: DEBUG: cluster ccm directory: /tmp/dtest-U0pP7H > dtest: DEBUG: Creating connection pools.. > dtest: DEBUG: Disabling/Enabling thrift iteration #0 > dtest: DEBUG: Closing connections from the client side.. > pycassa.pool: INFO: Pool 139721997822224 was disposed > pycassa.pool: INFO: Pool 139721997714512 was disposed > pycassa.pool: INFO: Pool 139721997749904 was disposed > --------------------- >> end captured logging << --------------------- > Stacktrace > File "/usr/lib/python2.7/unittest/case.py", line 331, in run > testMethod() > File "/home/automaton/cassandra-dtest/thrift_hsha_test.py", line 59, in > test_closing_connections > self.assertEqual(len(lines), 0, "There are non-closed connections: %s" % > stdout) > File "/usr/lib/python2.7/unittest/case.py", line 515, in assertEqual > assertion_func(first, second, msg=msg) > File "/usr/lib/python2.7/unittest/case.py", line 508, in _baseAssertEqual > raise self.failureException(msg) > 'There are non-closed connections: COMMAND PID USER FD TYPE DEVICE > SIZE/OFF NODE NAME\njava 28202 automaton 7u IPv4 60767 0t0 TCP > localhost:58436->localhost:9160 (CLOSE_WAIT)\n\n-------------------- >> begin > captured logging << --------------------\ndtest: DEBUG: cluster ccm > directory: /tmp/dtest-U0pP7H\ndtest: DEBUG: Creating connection > pools..\ndtest: DEBUG: Disabling/Enabling thrift iteration #0\ndtest: DEBUG: > Closing connections from the client side..\npycassa.pool: INFO: Pool > 139721997822224 was disposed\npycassa.pool: INFO: Pool 139721997714512 was > disposed\npycassa.pool: INFO: Pool 139721997749904 was > disposed\n--------------------- >> end captured logging << > ---------------------' > {noformat} -- This message was sent by Atlassian JIRA (v6.3.4#6332)