binarycat0 opened a new issue, #2487: URL: https://github.com/apache/polaris/issues/2487
The `Converter` interface is a bit strange indeed. The method above is a pure converter utility, but this one looks like it's meant to convert the object's _internal state_ to some external representation. This hints at the idea that this interface should be split in two. In `ResultSetIterator` for instance, we only need the `fromResultSet` method. _Originally posted by @adutra in https://github.com/apache/polaris/pull/2465#discussion_r2315955845_ -- 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]
