Re: [PR] open-api: Add namespaceExist API [iceberg]

2023-10-02 Thread via GitHub
dramaticlly commented on PR #8569: URL: https://github.com/apache/iceberg/pull/8569#issuecomment-1743610048 thank you @Fokko, @amogh-jahagirdar and @nastra for the review! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] open-api: Add namespaceExist API [iceberg]

2023-10-02 Thread via GitHub
Fokko commented on PR #8569: URL: https://github.com/apache/iceberg/pull/8569#issuecomment-1743609595 Fair point, let's add this. Thanks @nastra and @amogh-jahagirdar for the reviews, and @dramaticlly for raising this PR! -- This is an automated message from the Apache Git Service. To res

Re: [PR] open-api: Add namespaceExist API [iceberg]

2023-10-02 Thread via GitHub
Fokko merged PR #8569: URL: https://github.com/apache/iceberg/pull/8569 -- 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.apach

Re: [PR] open-api: Add namespaceExist API [iceberg]

2023-10-02 Thread via GitHub
dramaticlly commented on PR #8569: URL: https://github.com/apache/iceberg/pull/8569#issuecomment-1743604553 > Sorry for being so late to the party. I don't have a strong objection against this, but I'm curious about what the use case is. If you want to create a namespace then it is best to

Re: [PR] open-api: Add namespaceExist API [iceberg]

2023-10-02 Thread via GitHub
Fokko commented on PR #8569: URL: https://github.com/apache/iceberg/pull/8569#issuecomment-1743585167 Sorry for being so late to the party. I don't have a strong objection against this, but I'm curious about what the use case is. If you want to create a namespace then it is best to just try