eric-wang-1990 opened a new pull request, #3304: URL: https://github.com/apache/arrow-adbc/pull/3304
# Add Environment Variable Configuration Support for Databricks Driver ## Overview Introduces robust configuration loading from JSON files via environment variables with intelligent property merging and configurable precedence control. ## Key Features ### 🔧 Environment Variable Configuration - **Default environment variable**: `DATABRICKS_CONFIG_FILE` automatically loaded in constructor - **Standard ADBC parameter**: The keys are standard adbc parameter: https://github.com/apache/arrow-adbc/blob/main/csharp/src/Drivers/Databricks/readme.md ```json { "adbc.databricks.driver_config_take_precedence": "true", "adbc.databricks.enable_pk_fk": "false" } ``` ### 🎛️ Flexible Property Merging - **Automatic merging**: Environment config + constructor properties - **Configurable precedence**: New `adbc.databricks.driver_config_take_precedence` parameter - **Smart defaults**: Constructor properties override environment config by default ## Backward Compatibility - ✅ All existing code works unchanged - ✅ Original constructor behavior preserved - ✅ Same exception patterns maintained - ✅ Constructor properties still take precedence by default -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
