amansinha100 commented on a change in pull request #729: Drill 1328: Support 
table statistics for Parquet
URL: https://github.com/apache/drill/pull/729#discussion_r250700249
 
 

 ##########
 File path: 
exec/java-exec/src/main/java/org/apache/drill/exec/util/Utilities.java
 ##########
 @@ -60,9 +60,12 @@ public static String 
getFileNameForQueryFragment(FragmentContext context, String
    * QueryContextInformation is derived from the current state of the process.
    *
    * @param defaultSchemaName
+   * @param sessionId
+   * @param manager
    * @return A {@link 
org.apache.drill.exec.proto.BitControl.QueryContextInformation} with given 
<i>defaultSchemaName</i>.
    */
-  public static QueryContextInformation createQueryContextInfo(final String 
defaultSchemaName, final String sessionId) {
+  public static QueryContextInformation createQueryContextInfo(final String 
defaultSchemaName,
+      final String sessionId, final OptionManager manager) {
 
 Review comment:
   The OptionManager is not used in this constructor ?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to