[GitHub] [guacamole-client] necouchman commented on a change in pull request #546: GUACAMOLE-1123: Standardize REST endpoints for historical data

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

necouchman commented on a change in pull request #546:
URL: https://github.com/apache/guacamole-client/pull/546#discussion_r475256938



##########
File path: guacamole-ext/src/main/java/org/apache/guacamole/net/auth/ActivityRecordSet.java
##########
@@ -56,6 +56,24 @@
      *      If an error occurs while retrieving the records within this set.
      */
     Collection<RecordType> asCollection() throws GuacamoleException;
+    
+    /**
+     * Returns records within this set for the item having the specified
+     * identifier as a standard Collection.
+     *
+     * @param identifier
+     *     The identifier of the underlying item that the collection
+     *     should be limited to.
+     * 
+     * @return
+     *     A collection containing records within this set for the specified
+     *     identifier.
+     *
+     * @throws GuacamoleException
+     *     If an error occurs while retrieving the records within this set.
+     */
+    Collection<RecordType> asCollection(String identifier)
+            throws GuacamoleException;

Review comment:
       Okay, I have taken a wild stab at this - let me know how it looks, now.




----------------------------------------------------------------
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:
users@infra.apache.org