leesf commented on a change in pull request #2714:
URL: https://github.com/apache/hudi/pull/2714#discussion_r624690373
##########
File path:
hudi-common/src/main/java/org/apache/hudi/common/table/view/AbstractTableFileSystemView.java
##########
@@ -271,15 +271,15 @@ private void ensurePartitionLoadedCorrectly(String
partition) {
if (!isPartitionAvailableInStore(partitionPathStr)) {
// Not loaded yet
try {
- LOG.info("Building file system view for partition (" +
partitionPathStr + ")");
+ LOG.debug("Building file system view for partition (" +
partitionPathStr + ")");
// Create the path if it does not exist already
Path partitionPath =
FSUtils.getPartitionPath(metaClient.getBasePath(), partitionPathStr);
FSUtils.createPathIfNotExists(metaClient.getFs(), partitionPath);
long beginLsTs = System.currentTimeMillis();
FileStatus[] statuses = listPartition(partitionPath);
long endLsTs = System.currentTimeMillis();
- LOG.info("#files found in partition (" + partitionPathStr + ") =" +
statuses.length + ", Time taken ="
+ LOG.debug("#files found in partition (" + partitionPathStr + ") =" +
statuses.length + ", Time taken ="
Review comment:
this would be `info` level ? it is helpful to show the time cost.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
