Eduard Tudenhoefner
Eduard Tudenhoefner
`"default-namespace": []` indicates an empty namespace, not a null one, which is a valid case and there are tests for that here: https://github.com/apache/iceberg/blob/42a2c19cec31c626cbff6cc2dfafb86cdf223bd0/core/src/test/java/org/apache/iceberg/view/TestViewMetadataParser.java#L56. I've also opened https://github.com/apache/iceberg/pull/9890 a while ago...
Ok I checked the surrounding code and the handling is correct. In https://github.com/apache/iceberg/blob/2a39af894f4f00aa37922ef765cc2583517fa1d1/spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/execution/datasources/v2/CreateV2ViewExec.scala#L51 we're always determining the current namespace (which can be empty), which is then what `ViewVersion#defaultNamespace` is set...
@jackye1995 should we maybe first get https://github.com/apache/iceberg-docs/pull/187 in? @gaborkaszab was doing the previous release and also added a few good items there
@amogh-jahagirdar is this still relevant?
@VeeraswamyGatta I don't think Azure is currently supported by the Iceberg destination. Also it's not in the scope of this PR to add Azure support
> @nastra thanks for the contribution. I need to check the current integration tests for destination iceberg, I think they're broken today. @marcosmarxm they were all passing for me locally...
> @nastra > > Thank you. If I want to include Azure support for this connector, do you have any suggestions on how to complete the Azure support? @VeeraswamyGatta you...
@adutra I'm OOO this week, maybe @amogh-jahagirdar gets a chance to look at this PR
@adutra IMO https://github.com/apache/iceberg/pull/10314/files#r1601220154 still needs to be reverted. It's ok to add a workaround for Keycloak but I don't see a good enough reason to adjust what the server (aka...
@adutra I've read RFC 6749 but as I mentioned in https://github.com/apache/iceberg/pull/10314#discussion_r1601201951 I'm assuming that the token exchange is following RFC 8693 (and my assumption can be wrong). It would be...