Add create_sql_view method to Catalog

[ebyhr]

Rationale for this change

PyIceberg users currently need to provide a redundant object when creating a view.

dentifier = "default.some_view"
schema = pa.schema([pa.field("some_col", pa.int32())])
view_version = ViewVersion(
    version_id=1,
    schema_id=1,
    timestamp_ms=int(time.time() * 1000),
    summary={},
    representations=[
        SQLViewRepresentation(
            type="sql",
            sql="SELECT 1 as some_col",
            dialect="spark",
        )
    ],
    default_namespace=["default"],
)

catalog.create_view(
    identifier=identifier,
    schema=schema,
    view_version=view_version,
)

We could internally initialize version_id, schema_id, and timestamp_ms fields. This PR introduces a new create_sql_view method.

catalog.create_sql_view(
    identifier=identifier,
    schema=schema,
    dialect="spark",
    sql="SELECT 1 as some_col",
    default_namespace=("default",),
)

Are these changes tested?

Yes

Are there any user-facing changes?

Add create_sql_view method to Catalog

[gabeiglio]

lgtm!

[rambleraptor]

I'm lightly opinionated on making this a helper function that takes in a Catalog, rather than a new method on the Catalog itself.

[abnobdoss]

This looks good! Left a few minor comments.

[abnobdoss]

Suggested change

	schema (Schema): View's schema.
	schema (Schema | pa.Schema): View's schema.

[abnobdoss]

Should this description be different from create_view?

[abnobdoss]

is this schema_id discarded later in the self.create_view call later?

iceberg-python/pyiceberg/catalog/rest/__init__.py

Line 981 in c85c9ee

fresh_schema = assign_fresh_schema_ids(iceberg_schema)

[abnobdoss]

The create_sql_view function has a few permutations of accepted values - e.g. default_namespace accepts both str & Identifier. Could we expand the test coverage to cover the various signature paths?

[github-actions]

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.