From 998d200618f6c6699f8bacd059ffda7c54f4d39e Mon Sep 17 00:00:00 2001
From: Piotr Findeisen <piotr.findeisen@gmail.com>
Date: Fri, 16 Mar 2018 09:14:00 +0100
Subject: [PATCH] Fix grammar in PreconditionRules's method name

Also, make the helper method private.
---
 .../presto/sql/planner/iterative/rule/PreconditionRules.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/presto-main/src/main/java/com/facebook/presto/sql/planner/iterative/rule/PreconditionRules.java b/presto-main/src/main/java/com/facebook/presto/sql/planner/iterative/rule/PreconditionRules.java
index b6f68a197c1a..fb7a05278844 100644
--- a/presto-main/src/main/java/com/facebook/presto/sql/planner/iterative/rule/PreconditionRules.java
+++ b/presto-main/src/main/java/com/facebook/presto/sql/planner/iterative/rule/PreconditionRules.java
@@ -28,10 +28,10 @@ private PreconditionRules() {}
 
     public static Rule<ExchangeNode> checkRulesAreFiredBeforeAddExchangesRule()
     {
-        return checkPlanDoNotMatch(exchange(), "Expected rules to be fired before 'AddExchanges' optimizer");
+        return checkNoPlanNodeMatches(exchange(), "Expected rules to be fired before 'AddExchanges' optimizer");
     }
 
-    public static <T extends PlanNode> Rule<T> checkPlanDoNotMatch(Pattern<T> pattern, String message)
+    private static <T extends PlanNode> Rule<T> checkNoPlanNodeMatches(Pattern<T> pattern, String message)
     {
         return new CheckNoPlanNodeMatchesRule<>(pattern, message);
     }