[View Support] Add ViewMetadata read support and create_view to REST Catalog

[rambleraptor]

Part of #818

This adds create_view to the REST Catalog. As part of this, it also defines the View and ViewMetadata class.

Rationale for this change

PyIceberg's REST Catalog doesn't support create_view. Furthermore, PyIceberg doesn't support views at all. This is the first part in getting views supported.

Are these changes tested?

Unit tests included

Are there any user-facing changes?

Added Iceberg REST Catalog support for create_view

[jayceslesar]

Is it worth setting integration tests in rest integration tests?

[rambleraptor]

I'm a little unclear what Catalog the REST integration tests run against, so I can't say if there's been enough implemented to actually have a valid REST Catalog test.

The integration tests are missing most of the methods that have been implemented, so my reaction is that we're still missing pieces for an integration test.

[rambleraptor]

@Fokko@kevinjqliu mind taking a look when you can? thanks!

[sungwy]

Thanks for taking this up @rambleraptor

I've taken a stab at reviewing the PR. I think adding an integration test is a must-have for introducing a new API. I think a great way to verify that the created view works as expected is to add a view as a spark dialect sql view and read the view table through a spark session, similar to how we validate that PyIceberg writes are reflected correctly even when the table is read through a Spark session accessing the same REST Catalog (example). WDYT?

[rambleraptor]

@sungwy@jayceslesar please take a look when you can!

[rambleraptor]

addressed the comments, thanks for the review!

[sungwy]

Hi @rambleraptor the CI is failing due to a lint error.

pyiceberg/view/__init__.py:23:22: F401 `pydantic.Field` imported but unused
|
21 | )
22 |
23 | from pydantic import Field
| ^^^^^ F401
24 |
25 | from pyiceberg.typedef import Identifier
|
= help: Remove unused import: `pydantic.Field`

Could you run make lint and commit the changes?

[rambleraptor]

@sungwy whoops, thanks! all fixed.

[rambleraptor]

@sungwy@jayceslesar@Fokko can I get a review on this when you can? Thanks!

[sjdurfey]

is this issue dead?

[rambleraptor]

@sjdurfey I just updated the PR to handle the merge conflicts. I'm definitely interested in getting more View support in and I think this PR is a good place to begin!

@kevinjqliu@Fokko fyi, ptal!

[Fokko]

Left some small comments, but this looks good to me 👍

[rambleraptor]

@Fokko changes made! On vacation the last week, thanks for being patient!

[Fokko]

Hope you had a great vacation @rambleraptor, and thanks for working on this!