[GitHub] [hudi] parisni commented on a diff in pull request #6506: Allow hoodie read client to choose index

[GitBox <gi...@apache.org>]

parisni commented on code in PR #6506:
URL: https://github.com/apache/hudi/pull/6506#discussion_r960339900


##########
hudi-client/hudi-spark-client/src/main/java/org/apache/hudi/client/HoodieReadClient.java:
##########
@@ -92,6 +92,18 @@ public HoodieReadClient(HoodieSparkEngineContext context, String basePath, SQLCo
     this.sqlContextOpt = Option.of(sqlContext);
   }
 
+  /**
+   * @param context
+   * @param basePath
+   * @param sqlContext
+   * @param indexType
+   */
+  public HoodieReadClient(HoodieSparkEngineContext context, String basePath, SQLContext sqlContext, HoodieIndex.IndexType indexType) {

Review Comment:
   We use this client in our custom code. AFAIK this is the only way to benefit from bloom indexes while reading a hudi table. Then both global  bloom / bloom are useful 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org