kbendick opened a new pull request #4405:
URL: https://github.com/apache/iceberg/pull/4405


   Adds three empty interfaces, that are used for marking classes that 
correspond to REST requests / responses.
   1) `RESTMessage` - This marks either a RESTRequest or a RESTResponse. The 
name is chosen to match many libraries, like Netty, where the `HTTPRequest` 
extends `HTTPMessage`.
   2) `RESTRequest` - Specific empty subinterface for classes that correlate to 
REST catalog requests.
   3) `RESTResponse` - Specific empty subinterface for classes that correlate 
to REST catalog resposnes.
   
   Right now, these are empty but in the future we could add things like the 
`validate` method, or anything else that is needed. Eventually we could 
possibly even move these into the api module.
   
   For now, this is in response to [a comment 
here](https://github.com/apache/iceberg/pull/4245#discussion_r832570043) so 
that we can specify that the generic type going into a method is in fact a REST 
catalog request and that what's returned is a REST catalog response.
   
   cc @rdblue 


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to