wangxianghu commented on code in PR #19077:
URL: https://github.com/apache/hudi/pull/19077#discussion_r3498214262


##########
hudi-timeline-service/src/test/java/org/apache/hudi/timeline/service/functional/TestRemoteHoodieTableFileSystemView.java:
##########
@@ -112,6 +112,23 @@ public void testRemoteHoodieTableFileSystemViewWithRetry() 
{
     viewWithRetries.getLatestBaseFiles();
   }
 
+  /**
+   * The base class probes {@code AbstractTableFileSystemView.bootstrapIndex} 
via reflection to
+   * verify non-bootstrap tables get NoOpBootstrapIndex (avoiding S3 HEAD 
probes during init).
+   * {@link RemoteHoodieTableFileSystemView} does not extend {@code 
AbstractTableFileSystemView} —
+   * the real view, and hence that field, lives server-side, so the assertion 
isn't applicable here.
+   */
+  @Test
+  public void testBootstrapIndexIsNoOpWhenBootstrapBasePathAbsent() throws 
Exception {
+    // no-op: covered by the base test against the local view

Review Comment:
   done. thanks for your time @wombatu-kun 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to