hashhar commented on code in PR #11041:
URL: https://github.com/apache/iceberg/pull/11041#discussion_r2048363317


##########
format/view-spec.md:
##########
@@ -160,6 +179,57 @@ Each entry in `version-log` is a struct with the following 
fields:
 | _required_  | `timestamp-ms` | Timestamp when the view's 
`current-version-id` was updated (ms from epoch) |
 | _required_  | `version-id`   | ID that `current-version-id` was set to |
 
+#### Storage Table Identifier
+
+The table identifier for the storage table that stores the precomputed results.
+
+| Requirement | Field name     | Description |
+|-------------|----------------|-------------|
+| _optional_  | `catalog`      | A string specifying the name of the catalog. 
If set to `null`, the catalog is the same as the view's catalog |

Review Comment:
   What are some reasons why the catalog and namespace should not be inherited 
from the MV's catalog and namespace?
   
   IIUC the proposal allows:
   
   - catalog1.ns1.source_table (source table)
   - catalog2.ns2.mv (materialized view)
     - catalog3.ns3.storage_table (storage table)



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to