cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dave Brosius (JIRA)" <>
Subject [jira] [Comment Edited] (CASSANDRA-13106) Unnecessary assertion
Date Fri, 06 Jan 2017 01:06:58 GMT


Dave Brosius edited comment on CASSANDRA-13106 at 1/6/17 1:06 AM:

if you remove the -ea flag, this code will not be executed, and thus this issue will disappear.

was (Author: dbrosius):
if you remove the -ea flag, this issue will disappear.

> Unnecessary assertion
> ---------------------
>                 Key: CASSANDRA-13106
>                 URL:
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Simon Zhou
>            Assignee: Simon Zhou
>            Priority: Minor
>         Attachments: CASSANDRA-13106.patch
> We had over 70 thousand sstables and it's slow to bootstrap new node, even though the
CPU utilization for main thread of Cassandra was nearly 100%. So we took a few stack traces
and found that the main thread were busy running this line in
> {code}
> assert Iterables.all(removed, remove);
> {code}
> Not exactly sure whether this line causes CPU utilization/bootstrapping issue, but this
line is redundant because the Predict we pass in is Predicates.<SSTableReader>alwaysTrue(),
which means the assertion always returns true. So I propose to remove that line.

This message was sent by Atlassian JIRA

View raw message