Add logging to track latency for metrics as dimensions in query parsing #2000
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: MetricFlow SQL Engine Tests | |
| # Runs tests against the other SQL engines that are supported. These engines are hosted externally, and tend to be | |
| # slower. As the test will be I/O bound, the tests can be run with higher parallelism to get faster runtimes. | |
| on: | |
| workflow_dispatch: | |
| pull_request: | |
| types: [labeled] | |
| env: | |
| PYTHON_VERSION: "3.8" | |
| EXTERNAL_ENGINE_TEST_PARALLELISM: 8 | |
| ADDITIONAL_PYTEST_OPTIONS: "--use-persistent-source-schema" | |
| jobs: | |
| snowflake-tests: | |
| environment: DW_INTEGRATION_TESTS | |
| if: ${{ github.event.action != 'labeled' || github.event.label.name == 'Run Tests With Other SQL Engines' }} | |
| name: Snowflake Tests | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Check-out the repo | |
| uses: actions/checkout@v3 | |
| - name: Test w/ Python ${{ env.PYTHON_VERSION }} | |
| uses: ./.github/actions/run-mf-tests | |
| with: | |
| python-version: ${{ env.PYTHON_VERSION }} | |
| mf_sql_engine_url: ${{ secrets.MF_SNOWFLAKE_URL }} | |
| mf_sql_engine_password: ${{ secrets.MF_SNOWFLAKE_PWD }} | |
| parallelism: ${{ env.EXTERNAL_ENGINE_TEST_PARALLELISM }} | |
| additional-pytest-options: ${{ env.ADDITIONAL_PYTEST_OPTIONS }} | |
| make-target: "test-snowflake" | |
| redshift-tests: | |
| environment: DW_INTEGRATION_TESTS | |
| name: Redshift Tests | |
| if: ${{ github.event.action != 'labeled' || github.event.label.name == 'Run Tests With Other SQL Engines' }} | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Check-out the repo | |
| uses: actions/checkout@v3 | |
| - name: Test w/ Python ${{ env.PYTHON_VERSION }} | |
| uses: ./.github/actions/run-mf-tests | |
| with: | |
| python-version: ${{ env.PYTHON_VERSION }} | |
| mf_sql_engine_url: ${{ secrets.MF_REDSHIFT_URL }} | |
| mf_sql_engine_password: ${{ secrets.MF_REDSHIFT_PWD }} | |
| parallelism: ${{ env.EXTERNAL_ENGINE_TEST_PARALLELISM }} | |
| additional-pytest-options: ${{ env.ADDITIONAL_PYTEST_OPTIONS }} | |
| make-target: "test-redshift" | |
| bigquery-tests: | |
| environment: DW_INTEGRATION_TESTS | |
| name: BigQuery Tests | |
| if: ${{ github.event.action != 'labeled' || github.event.label.name == 'Run Tests With Other SQL Engines' }} | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Check-out the repo | |
| uses: actions/checkout@v3 | |
| - name: Test w/ Python ${{ env.PYTHON_VERSION }} | |
| uses: ./.github/actions/run-mf-tests | |
| with: | |
| python-version: ${{ env.PYTHON_VERSION }} | |
| MF_SQL_ENGINE_URL: ${{ secrets.MF_BIGQUERY_URL }} | |
| MF_SQL_ENGINE_PASSWORD: ${{ secrets.MF_BIGQUERY_PWD }} | |
| parallelism: ${{ env.EXTERNAL_ENGINE_TEST_PARALLELISM }} | |
| additional-pytest-options: ${{ env.ADDITIONAL_PYTEST_OPTIONS }} | |
| make-target: "test-bigquery" | |
| databricks-tests: | |
| environment: DW_INTEGRATION_TESTS | |
| name: Databricks Tests | |
| if: ${{ github.event.action != 'labeled' || github.event.label.name == 'Run Tests With Other SQL Engines' }} | |
| runs-on: ubuntu-latest | |
| steps: | |
| - name: Check-out the repo | |
| uses: actions/checkout@v3 | |
| - name: Test w/ Python ${{ env.PYTHON_VERSION }} | |
| uses: ./.github/actions/run-mf-tests | |
| with: | |
| python-version: ${{ env.PYTHON_VERSION }} | |
| mf_sql_engine_url: ${{ secrets.MF_DATABRICKS_URL }} | |
| mf_sql_engine_password: ${{ secrets.MF_DATABRICKS_PWD }} | |
| parallelism: ${{ env.EXTERNAL_ENGINE_TEST_PARALLELISM }} | |
| additional-pytest-options: ${{ env.ADDITIONAL_PYTEST_OPTIONS }} | |
| make-target: "test-databricks" | |
| trino-tests: | |
| # Trino tests run on a local service container, which obviates the need for separate Environment hosting. | |
| # We run them here instead of in the CI unit test suite because they are a bit slower, and because in future | |
| # we may choose to execute them against a hosted instance, at which point this config will look like the other | |
| # engine configs in this file. | |
| name: Trino Tests | |
| if: ${{ github.event.action != 'labeled' || github.event.label.name == 'Run Tests With Other SQL Engines' }} | |
| runs-on: ubuntu-latest | |
| services: | |
| trino: | |
| image: trinodb/trino | |
| ports: | |
| - 8080:8080 | |
| steps: | |
| - name: Check-out the repo | |
| uses: actions/checkout@v3 | |
| - name: Test w/ Python 3.11 | |
| uses: ./.github/actions/run-mf-tests | |
| with: | |
| python-version: "3.11" | |
| make-target: "test-trino" | |
| remove-label: | |
| name: Remove Label After Running Tests | |
| runs-on: ubuntu-latest | |
| needs: [ snowflake-tests, redshift-tests, bigquery-tests, databricks-tests ] | |
| if: github.event.action == 'labeled' && github.event.label.name == 'Run Tests With Other SQL Engines' | |
| steps: | |
| - name: Remove Label | |
| uses: actions-ecosystem/action-remove-labels@v1 | |
| with: | |
| labels: 'Run Tests With Other SQL Engines' |