Github user HanumathRao commented on a diff in the pull request:

    https://github.com/apache/drill/pull/996#discussion_r146118332
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/SchemaUtilites.java
 ---
    @@ -77,6 +77,22 @@ public static SchemaPlus findSchema(final SchemaPlus 
defaultSchema, final String
         return findSchema(defaultSchema, schemaPathAsList);
       }
     
    +  /**
    +   * Utility function to get the commonPrefix schema between two supplied 
schemas.
    +   * @param defaultSchema default schema
    +   * @param schemaPath current schema path
    +   * @param isCaseSensitive true if caseSensitive comparision is required.
    +   * @return common prefix schemaPath
    +   */
    +  public static String getPrefixSchemaPath(final String defaultSchema, 
final String schemaPath, final boolean isCaseSensitive) {
    --- End diff --
    
    @paul-rogers  Thanks for the review. For this API I am assuming the caller 
will supply the information whether it needs to be a case sensitive or case 
insensitive comparison. The caller for this function is passing the 
parserConfig case sensitive parameter.


---

Reply via email to