rdblue commented on a change in pull request #1920:
URL: https://github.com/apache/iceberg/pull/1920#discussion_r552136546
##########
File path: core/src/main/java/org/apache/iceberg/BaseTable.java
##########
@@ -29,8 +30,12 @@
* Base {@link Table} implementation.
* <p>
* This can be extended by providing a {@link TableOperations} to the
constructor.
+ * <p>
+ * Serializing and deserializing a BaseTable object returns a read only
implementation of the BaseTable using a
+ * {@link StaticTableOperations}. This way no Catalog related calls are needed
when reading the table data after
+ * deserialization.
*/
-public class BaseTable implements Table, HasTableOperations {
+public class BaseTable implements Table, HasTableOperations, Serializable {
Review comment:
I'm also wondering if we should make `Table` extend `Serializable`. Not
something to decide now, but I think this is a reasonable way to make that work.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
