Github user jasobrown commented on a diff in the pull request:

    https://github.com/apache/cassandra-dtest/pull/15#discussion_r159245703
  
    --- Diff: repair_tests/incremental_repair_test.py ---
    @@ -168,9 +171,10 @@ def _make_fake_session(self, keyspace, table):
             for node in self.cluster.nodelist():
                 session = self.patient_exclusive_cql_connection(node)
                 session.execute("INSERT INTO system.repairs "
    -                            "(parent_id, cfids, coordinator, last_update, 
participants, ranges, repaired_at, started_at, state) "
    -                            "VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s)",
    -                            [session_id, {cfid}, node1.address(), now, 
{n.address() for n in self.cluster.nodelist()},
    +                            "(parent_id, cfids, coordinator, 
coordinator_port, last_update, participants, participants_wp, ranges, 
repaired_at, started_at, state) "
    +                            "VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s, 
%s, %s)",
    +                            [session_id, {cfid}, node1.address(), 7000, 
now, {n.address() for n in self.cluster.nodelist()},
    +                             {str(n.address()) + ":7000" for n in 
self.cluster.nodelist()},
    --- End diff --
    
    is hardcoding a port of `7000` ok here?


---

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

Reply via email to