Skip to content

Improve error handling in quickstart setup script#3288

Merged
dimas-b merged 1 commit intoapache:mainfrom
tmater:quickstart_setup_error_handling
Dec 23, 2025
Merged

Improve error handling in quickstart setup script#3288
dimas-b merged 1 commit intoapache:mainfrom
tmater:quickstart_setup_error_handling

Conversation

@tmater
Copy link
Contributor

@tmater tmater commented Dec 17, 2025

Summary

Enhances the quickstart Docker Compose setup script with proper error detection and validation for all API calls. Each curl command now fails fast with clear error messages when requests fail or return invalid responses, making it easier to debug setup issues.

Testing

Manually tested the error handling by running the script with ALLOW_SETTING_S3_ENDPOINTS=false and validated that failures are properly detected and displayed:

polaris-setup-1  | ❌ Failed to create catalog
polaris-setup-1  | curl: (22) The requested URL returned error: 400
polaris-setup-1  | {"error":{"message":"Explicitly setting S3 endpoints is not allowed.","type":"IllegalArgumentException","code":400}}

Add error detection and validation to API calls in the quickstart
docker-compose setup with clear error messages for easier debugging.
@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Dec 17, 2025
@dimas-b dimas-b merged commit ed851c3 into apache:main Dec 23, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Dec 23, 2025
@dimas-b
Copy link
Contributor

dimas-b commented Dec 23, 2025

Thanks, @tmater !

evindj pushed a commit to evindj/polaris that referenced this pull request Jan 26, 2026
Add error detection and validation to API calls in the quickstart
docker-compose setup with clear error messages for easier debugging.
snazy added a commit to snazy/polaris that referenced this pull request Feb 11, 2026
* Disable renovate bot for openapi generator cli (apache#3306)

* Fix openapi-generator-cli version in build system

* Fix openapi-generator-cli version in build system

* Build: Ensure reproducible .properties files (apache#3089)

This is a safety net in case Properties are generated anywhere during the build.

* Generate release vote e-mail as Github Step Summary (apache#3150)

* Improve error handling in quickstart setup script (apache#3288)

Add error detection and validation to API calls in the quickstart
docker-compose setup with clear error messages for easier debugging.

* (doc): Fix tools doc and add tool doc for mcp server (apache#3311)

* chore(deps): update actions/stale digest to a21a081 (apache#3326)

* Add cancel release candidate workflow (apache#3321)

This commit fixes apache#3080

* fix(deps): update quarkus platform and group to v3.30.5 (apache#3329)

* chore(deps): update actions/checkout digest to 8e8c483 (apache#3319)

* fix(deps): update dependency org.agrona:agrona to v2.4.0 (apache#3333)

* (feat): Helm: add priority class name support for helm (apache#3310)

* Add priority class name support for helm

* Update changelog

* Update default port for metrics from 8282 to 8182 (apache#3335)

Corrects the default metrics port from 8282 to 8182 and clarifies the available metrics endpoints in the documentation.

* fix(deps): update dependency software.amazon.awssdk:bom to v2.40.16 (apache#3336)

* Add KMS options to catalogs create CLI (apache#3330)

* Add Polaris blog about KMS (apache#3331)

* Add Polaris blog about KMS

Following up on apache#2802

* Use mainstream PostgreSQLContainer (apache#3345)

Migrate from the deprecated PostgreSQLContainer class to the
mainstream one as suggested by javadoc.

* fix(deps): update dependency org.apache.iceberg:iceberg-bom to v1.10.1 (apache#3317)

* Enforce mypy for CLI (apache#3305)

* fix(deps): update dependency software.amazon.awssdk:bom to v2.41.1 (apache#3351)

* Last merged commit e75eb4b

---------

Co-authored-by: Yong Zheng <yongzheng0809@gmail.com>
Co-authored-by: Pierre Laporte <pierre@pingtimeout.fr>
Co-authored-by: Tamas Mate <50709850+tmater@users.noreply.github.com>
Co-authored-by: Mend Renovate <bot@renovateapp.com>
Co-authored-by: rishii-19-works <rishipandey1912@gmail.com>
Co-authored-by: Dmitri Bourlatchkov <dmitri.bourlatchkov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants