Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-03-07 Thread via GitHub
flyrain commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1985522848 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-28 Thread via GitHub
flyrain commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1976113681 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-27 Thread via GitHub
sungwy commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1974643193 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-27 Thread via GitHub
sungwy commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1974643193 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-27 Thread via GitHub
flyrain commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1974452617 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-27 Thread via GitHub
dimas-b commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1973840064 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-26 Thread via GitHub
sungwy commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1972692336 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-26 Thread via GitHub
flyrain commented on code in PR #12376: URL: https://github.com/apache/iceberg/pull/12376#discussion_r1972482202 ## open-api/rest-catalog-open-api.yaml: ## @@ -4566,7 +4568,9 @@ components: AuthenticationTimeoutResponse: description: -Credentials have timed

Re: [PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-25 Thread via GitHub
sungwy commented on PR #12376: URL: https://github.com/apache/iceberg/pull/12376#issuecomment-2683707993 > Thanks for the spec clarification, @sungwy ! The changes LGTM 👍 All thanks to you for the helpful reviews @dimas-b 💯 -- This is an automated message from the Apache Git Servic

[PR] OpenAPI: Use more clear language in recommending error responses [iceberg]

2025-02-22 Thread via GitHub
sungwy opened a new pull request, #12376: URL: https://github.com/apache/iceberg/pull/12376 The existence of the 419 AuthenticationTimeoutResponse status code caused confusion in the Iceberg community on the following two questions: - whether the error response should be issued by a REST