Skip to content

Commit

Permalink
Try fixing missing results name in the benchmark step (apache#9632)
Browse files Browse the repository at this point in the history
  • Loading branch information
gruuya authored Mar 16, 2024
1 parent 451d13a commit 6e90f01
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/pr_benchmarks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
with:
ref: refs/pull/${{ github.event.issue.number }}/head

- name: Setup data and generate unique result names
- name: Setup test data
# Workaround for `the input device is not a TTY`, appropriated from https://github.com/actions/runner/issues/241
shell: 'script -q -e -c "bash -e {0}"'
run: |
Expand All @@ -31,7 +31,8 @@ jobs:
# Setup the TPC-H data set with a scale factor of 10
./bench.sh data tpch
# Generate a unique-ish identifiers for the results
- name: Generate unique result names
run: |
echo "HEAD_LONG_SHA=$(git log -1 --format='%H')" >> "$GITHUB_ENV"
echo "HEAD_SHORT_SHA=$(git log -1 --format='%h' --abbrev=7)" >> "$GITHUB_ENV"
echo "BASE_SHORT_SHA=$(echo "${{ github.sha }}" | cut -c1-7)" >> "$GITHUB_ENV"
Expand Down

0 comments on commit 6e90f01

Please sign in to comment.