necouchman commented on pull request #546: URL: https://github.com/apache/guacamole-client/pull/546#issuecomment-682749518
Okay, should be cleaned up. I think the only other thing I'm unsure about at this point is whether passing around the `null` identifier is the best way to go. It keeps the code as lean as possible by leveraging the existing MyBatis classes and queries without the need for additional queries, and still leverages the database to filter the data (which should be the fastest/most efficient way to accomplish that), but passing around `null` feels a little strange. Is that okay, or should I take a different approach, there - like just pulling all the records from the DB and doing some filtering within a Java class? ---------------------------------------------------------------- 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: [email protected]
