fix: restore hydrateHistoryEntities hook after setFieldsInBulk and update javadoc

Agent-Logs-Url: https://github.com/open-metadata/OpenMetadata/sessions/6541c953-9da7-4b9b-9bca-3d3a4792516b

Co-authored-by: TeddyCr <13626425+TeddyCr@users.noreply.github.com>

Author: Copilot

openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/EntityRepository.java

@@ -2228,6 +2228,7 @@ public final ResultList<T> listEntityHistoryByTimestamp(
 
     List<T> entities = JsonUtils.readObjects(jsons, getEntityClass());
     setFieldsInBulk(putFields, entities);
+    hydrateHistoryEntities(entities);
 
     int total = getVersionCountCached(tableName, startTs, endTs, entityType);
 
@@ -2265,14 +2266,14 @@ public final ResultList<T> listEntityHistoryByTimestamp(
   }
 
   /**
-   * Hook to hydrate entities returned from {@link #listEntityHistoryByTimestamp(long, long, String,
-   * String, int)}.
+   * Hook called after {@link #setFieldsInBulk} for entities returned from {@link
+   * #listEntityHistoryByTimestamp(long, long, String, String, int)}.
    *
-   * <p>Default behavior is intentionally lightweight: return the historical snapshots as stored in
-   * the extension table and avoid expensive relationship re-hydration for each row.
+   * <p>Subclasses may override to perform additional, entity-specific hydration of history
+   * snapshots without overriding the core field-population logic in {@code setFieldsInBulk}.
    */
   protected void hydrateHistoryEntities(List<T> entities) {
-    // Historical snapshots are already serialized versions; avoid N+1 hydration on /history.
+    // No additional hydration by default.
   }
 
   private String decodeAndValidateCursor(String cursor) {