search

JenspederM / kedro-databricks

A Databricks Plugin for Kedro
MIT License
13 stars 5 forks source link

Refactor/init #64

Closed JenspederM closed 3 weeks ago

JenspederM commented 4 weeks ago

The changes in the pull request #64 include updates to several files:

  1. src/kedro_databricks/init.py:

    • Added InitController class to manage Databricks initialization, configuration, run script creation, and path substitution.
    • Introduced has_databricks_cli utility function to check if the Databricks CLI is installed.
    • Removed hardcoded templates and replaced them with resource-based templates.

  2. src/kedro_databricks/plugin.py:

    • Replaced direct function calls with methods from the InitController class.
    • Updated initialization logic to use the new InitController.

  3. src/kedro_databricks/utils.py:

    • Added has_databricks_cli function to check for Databricks CLI installation.

  4. tests/unit/test_init.py:

    • Updated tests to use InitController methods.
    • Added new tests to cover the functionality provided by InitController.
    • These changes refactor the initialization and configuration process for Databricks integration, adding a more structured and maintainable approach through the InitController class.
codeclimate[bot] commented 4 weeks ago

Code Climate has analyzed commit b5763f5e and detected 0 issues on this pull request.

View more on Code Climate.