[GitHub] [hudi] vinothchandar commented on a change in pull request #3901: [MINOR] Downloads from Nexus Pentaho repo taking too long

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

vinothchandar commented on a change in pull request #3901:
URL: https://github.com/apache/hudi/pull/3901#discussion_r740523866



##########
File path: hudi-client/hudi-spark-client/src/test/java/org/apache/hudi/client/functional/TestHoodieBackedMetadata.java
##########
@@ -834,6 +834,7 @@ public void testCleaningArchivingAndCompaction() throws Exception {
         .withCompactionConfig(HoodieCompactionConfig.newBuilder().archiveCommitsWith(2, 4)
             .withFailedWritesCleaningPolicy(HoodieFailedWritesCleaningPolicy.NEVER)
             .retainCommits(1).retainFileVersions(1).withAutoClean(true).withAsyncClean(false).build())
+        .withEmbeddedTimelineServerEnabled(false)

Review comment:
       why are we doing this change? this is turned on by default in the real world?

##########
File path: hudi-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadata.java
##########
@@ -199,10 +203,14 @@ private void initIfNeeded() {
           }
         } else {
           // only log record
+          LOG.warn("TEST_LOG. 111 Looking for record " + key + ", isPresent in log " + logRecords.containsKey(key));
           if (logRecords.containsKey(key) && logRecords.get(key).isPresent()) {
+            LOG.warn("TEST_LOG. 222 found log record " + key);
             HoodieRecordPayload mergedPayload = logRecords.get(key).get().getData().preCombine(hoodieRecord.getData());
+            LOG.warn("TEST_LOG. 3333 merged log record " + key);
             result.add(Pair.of(key, Option.of(new HoodieRecord(hoodieRecord.getKey(), mergedPayload))));
           } else { // not found in both base file and log files
+            LOG.warn("TEST_LOG. 444 not log record found " + key);

Review comment:
       revert all this?




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