jerryshao commented on code in PR #12891:
URL: https://github.com/apache/gravitino/pull/12891#discussion_r3931901663


##########
core/src/main/java/org/apache/gravitino/listener/api/event/Event.java:
##########
@@ -51,4 +69,34 @@ protected Event(String user, NameIdentifier identifier) {
   public String remoteAddress() {
     return remoteAddress;
   }
+
+  /**
+   * Returns the current request's (raw) query parameters merged with this 
event's own facts (from
+   * {@link #ownCustomInfo()}); the subclass's own values win on a key 
collision. This method is
+   * {@code final} — override {@link #ownCustomInfo()} to contribute 
subclass-specific facts.
+   *
+   * @return the merged custom info, or just the automatically captured query 
parameters if the
+   *     subclass contributes nothing of its own.
+   */
+  @Override
+  public final Map<String, String> customInfo() {

Review Comment:
   I think this is on-purpose. This expose the raw string for other custom 
listener, so they can do what they want at their risk.



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