[GitHub] [hudi] bvaradar commented on a diff in pull request #7323: [HUDI-5276][WIP] Exclude unnecessary partiton paths return

["bvaradar (via GitHub)" <gi...@apache.org>]

bvaradar commented on code in PR #7323:
URL: https://github.com/apache/hudi/pull/7323#discussion_r1121086268


##########
hudi-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadata.java:
##########
@@ -146,7 +146,7 @@ protected Option<HoodieRecord<HoodieMetadataPayload>> getRecordByKey(String key,
   @Override
   public List<String> getPartitionPathsWithPrefixes(List<String> prefixes) throws IOException {
     return getAllPartitionPaths().stream()
-        .filter(p -> prefixes.stream().anyMatch(p::startsWith))
+        .filter(p -> prefixes.stream().anyMatch(queryPaths -> p.startsWith(queryPaths + "/") || queryPaths.equals(p)))

Review Comment:
   @onlywangyh : Is the current behavior in Hive differs from Parquet Hive and Presto table ?  If so, can we change in the caller side (for e:g HiveHoodieTableFileIndex) instead of this place. 



-- 
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