rubenvdg opened a new pull request, #6644:
URL: https://github.com/apache/iceberg/pull/6644

   This PR proposes adding support for static tables (i.e., reading a table 
directly from a metadata file without using a catalog, see also 
https://github.com/apache/iceberg/issues/6430). Happy to hear if this makes 
sense.
   
   Regarding unit tests:
   We could run all the existing `Table` tests for `StaticTable` too, but that 
might be a bit artificial. You'd get something ugly like this:
   
   ```python
   @pytest.fixture
   def table(..) -> Table:
       ...
   
   @pytest.fixture
   def static_table(...) -> StaticTable:
       ...
   
   @pytest.mark.parametrize("the_table", ["table", "static_table"])
   def test_schema(the_table: Table, request: FixtureRequest) -> None:
       assert request.getfixturevalue(the_table).schema() == Schema(...)
   ```


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