Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2440219672 > look at how load_table works today: we return a Table model with all the metadata about the table, and this model exposes functionality for data scans, etc. So load_view wo

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-22 Thread via GitHub
corleyma commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2430157206 @shiv-io It should still be possible to do `load_view` _without_ supporting any scanning functionality yet, and like @sungwy says, that is likely a necessary precursor for othe

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-22 Thread via GitHub
shiv-io commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2429788539 I'm fairly new to the Iceberg ecosystem -- thanks for the insightful discussion, looks like I have some reading to do before I can weigh in. `load_view` aside though, I'd

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-22 Thread via GitHub
corleyma commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2429506003 I agree with @sungwy that the primary goal of pyiceberg should be to make it possible for query engines to interface with Iceberg tables and views. Nonetheless, it would be rea

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-22 Thread via GitHub
sungwy commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2429209404 That's an interesting question @corleyma . The way I see it, PyIceberg is a language library, that tries to remain open to any Python based query engine that wants to make use of

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-21 Thread via GitHub
ndrluis commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2428029460 Following what @danielcweeks said in [this email](https://lists.apache.org/thread/onbgy4onmbply31kg4fsfxz1t2g9w1qq), I believe we could discuss and experiment with SQLGlot to cr

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-21 Thread via GitHub
corleyma commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2427932973 I am really curious about how `Load View` should work, given that currently only SQL representations of views are supported and I don't think we have an in-process SQL engine t

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-20 Thread via GitHub
shiv-io commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2425171379 Would love to take a first stab at this @kevinjqliu, could you assign? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-06-15 Thread via GitHub
syun64 commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2170955774 Thank you for raising this @ndrluis 💯 I will add this as a 0.8.0 milestone for now -- This is an automated message from the Apache Git Service. To respond to the message, plea