Remove no-op assert statement in ClientQuerySegmentWalker (#9607)

* Remove no-op assert statement

The assert statement in ClientQuerySegmentWalker will always be true because
of the preceeding while loop which has the same condition.
This change removes dead code to fix an error reported by LGTM

* Suppress lgtm

* cleanup whitespace
This commit is contained in:
Suneet Saldanha 2020-04-10 10:41:29 -07:00 committed by GitHub
parent 642fe83897
commit bd1cff24a2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 2 deletions

View File

@ -274,8 +274,7 @@ public class ClientQuerySegmentWalker implements QuerySegmentWalker
current = Iterables.getOnlyElement(current.getChildren());
}
assert !(current instanceof QueryDataSource);
assert !(current instanceof QueryDataSource); // lgtm [java/contradictory-type-checks]
current = inlineIfNecessary(current, null, subqueryRowLimitAccumulator, maxSubqueryRows, dryRun);
while (!stack.isEmpty()) {