Bug fix: use FULL OUTER JOIN
for dimension-only queries
#2143
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: Required Tests | |
on: | |
pull_request: | |
# run these jobs when a PR is opened, reopened, or updated (synchronize) | |
# synchronize = commit(s) pushed to the pull request | |
types: | |
- opened | |
- reopened | |
- synchronize | |
jobs: | |
metricflow-unit-tests-duckdb: | |
name: MetricFlow Tests (DuckDB) | |
runs-on: ubuntu-latest | |
strategy: | |
matrix: | |
python-version: ["3.8", "3.11"] | |
steps: | |
- name: Check-out the repo | |
uses: actions/checkout@v3 | |
- name: Test w/ Python ${{ matrix.python-version }} | |
uses: ./.github/actions/run-mf-tests | |
with: | |
python-version: ${{ matrix.python-version }} | |
metricflow-unit-tests-postgres: | |
name: MetricFlow Tests (PostgreSQL) | |
runs-on: ubuntu-latest | |
services: | |
postgres: | |
image: postgres | |
env: | |
POSTGRES_USER: metricflow | |
POSTGRES_PASSWORD: metricflowing | |
POSTGRES_DB: metricflow | |
options: >- | |
--health-cmd pg_isready | |
--health-interval 10s | |
--health-timeout 5s | |
--health-retries 5 | |
ports: | |
- 5432:5432 | |
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-postgresql" | |
metricflow-unit-tests: | |
name: MetricFlow Unit Tests | |
runs-on: ubuntu-latest | |
if: ${{ always() }} | |
needs: [metricflow-unit-tests-duckdb, metricflow-unit-tests-postgres] | |
steps: | |
- name: Check success | |
run: test ${{ needs.metricflow-unit-tests-duckdb.result }} = 'success' -a ${{ needs.metricflow-unit-tests-postgres.result }} = 'success' |