[CALCITE-2342] use of assert with side-effect in Schemas

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.17.0
• Component/s: core
• Assignee: Laurent Goujon
• Reporter: Laurent Goujon
• Created: 31/May/18 03:07
• Updated: 20/Jul/18 08:18
• Resolved: 05/Jul/18 11:38

Description

Schemas#path(CalciteSchema,Iterable<String>) method has the following code:

if (!rootSchema.name.isEmpty()) {
  assert rootSchema.name.equals(iterator.next());
}

Depending if assertions are enabled or not, iterator state might be different

Issue Links

• links to: Pull Request