[CALCITE-1519] Standardize on "sub-query" rather than "subquery" in class names and comments

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Labels: sub-query
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 30/Nov/16 23:01
• Updated: 06/Jan/17 19:30
• Resolved: 14/Dec/16 16:48

Description

I know it's foolish consistency, but this one has been bugging me for a long time. Let's standardize on "sub-query" rather than "subquery" in class names and comments.

Here are the public and protected Java names that would change:

• public interface SubqueryConverter becomes SubQueryConverter
• public enum RelOptUtil.SubqueryType becomes SubQueryType
• public void SqlPrettyWriter.setSubqueryStyle() becomes setSubQueryStyle
• public void SqlToRelConverter.setSubqueryConverter() becomes setSubQueryConverter
• public boolean SqlToRelConverter.canConvertSubquery() becomes canConvertSubQuery
• public boolean SqlToRelConverter.convertSubquery() becomes convertSubQuery
• public SqlToRelConverter.Config.getInSubqueryThreshold() becomes getInSubQueryThreshold

These are breaking changes, but will be easy to identify and fix by just compiling the code, and the APIs are not widely used.

The following will change, but leave behind the old name deprecated until 2.0:

• public static final int SqlToRelConverter.DEFAULT_IN_SUBQUERY_THRESHOLD
• public ConfigBuilder ConfigBuilder.withInSubqueryThreshold
• public static Programs.subquery()
• SqlAbstractParserImpl.ACCEPT_SUBQUERY becomes ACCEPT_SUB_QUERY
• SqlAbstractParserImpl.ACCEPT_NONQUERY becomes ACCEPT_NON_QUERY

Also a whole bunch of test names will change, and private/package-protected members, but that should not affect anyone.

Issue Links

• links to: Dev branch