kaka11chen commented on code in PR #20679: URL: https://github.com/apache/doris/pull/20679#discussion_r1226267783
########## fe/fe-core/src/main/java/org/apache/doris/planner/external/HiveScanNode.java: ########## @@ -192,12 +195,50 @@ private void getFileSplitByPartitions(HiveMetaStoreCache cache, List<HivePartiti for (HiveMetaStoreCache.HiveFileStatus status : fileCacheValue.getFiles()) { allFiles.addAll(splitFile(status.getPath(), status.getBlockSize(), status.getBlockLocations(), status.getLength(), status.getModificationTime(), - isSplittable, fileCacheValue.getPartitionValues())); + isSplittable, fileCacheValue.getPartitionValues(), fileCacheValue.getAcidInfo())); } } } } + private List<Split> splitFile(Path path, long blockSize, BlockLocation[] blockLocations, long length, Review Comment: ok ########## fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerUtil.java: ########## @@ -114,17 +114,19 @@ public static String printBroker(String brokerName, TNetworkAddress address) { public static List<String> parseColumnsFromPath(String filePath, List<String> columnsFromPath) throws UserException { - return parseColumnsFromPath(filePath, columnsFromPath, true); + return parseColumnsFromPath(filePath, columnsFromPath, true, false); } public static List<String> parseColumnsFromPath( String filePath, List<String> columnsFromPath, - boolean caseSensitive) + boolean caseSensitive, + boolean isACID) throws UserException { if (columnsFromPath == null || columnsFromPath.isEmpty()) { return Collections.emptyList(); } + int pathCount = isACID ? 3 : 2; Review Comment: ok -- 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-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org