vburenin commented on pull request #2714: URL: https://github.com/apache/hudi/pull/2714#issuecomment-830821555
It is not helpful by any means if there are a few thousands partitions are getting updated every 15 minutes. Dumping aggregated stats would way more useful. Like p50, p95. On Sun, May 2, 2021 at 7:54 AM leesf ***@***.***> wrote: > ***@***.**** commented on this pull request. > ------------------------------ > > In > hudi-common/src/main/java/org/apache/hudi/common/table/view/AbstractTableFileSystemView.java > <https://github.com/apache/hudi/pull/2714#discussion_r624690396>: > > > @@ -293,7 +293,7 @@ private void ensurePartitionLoadedCorrectly(String partition) { > LOG.debug("View already built for Partition :" + partitionPathStr + ", FOUND is "); > } > long endTs = System.currentTimeMillis(); > - LOG.info("Time to load partition (" + partitionPathStr + ") =" + (endTs - beginTs)); > + LOG.debug("Time to load partition (" + partitionPathStr + ") =" + (endTs - beginTs)); > > ditto > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/hudi/pull/2714#pullrequestreview-649913490>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ABBGHK56VYKTU23WE4O3ZJLTLVDOTANCNFSM4ZXNOKDA> > . > -- 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]
