Skip to content

Commit

Permalink
refactor: (GH-#85) Changes in benchmark after review-2
Browse files Browse the repository at this point in the history
Signed-off-by: Saurabh Kumar Singh <[email protected]>
  • Loading branch information
singh1203 committed Feb 1, 2025
1 parent 1ea6db4 commit 4dc951e
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 9 deletions.
15 changes: 6 additions & 9 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,9 @@ name: Benchmarks
on:
push:
branches: [main]
pull_request:
branches: [main]
paths:
- ".github/workflows/benchmark.yml"
- "ci/scripts/bench_adapt.py"
workflow_dispatch:
permissions:
contents: read
Expand All @@ -40,19 +39,17 @@ jobs:
uses: actions/setup-python@v4
with:
python-version: '3.9'
- name: Install dependencies
run: |
apt-get update
apt-get install -y build-essential git python3-pip
- name: Install Go ${{ matrix.go }} for Benchmarks
uses: actions/setup-go@v5
with:
go-version: ${{ matrix.go }}
cache: true
cache-dependency-path: go.sum
- name: Run Benchmarks
if: >-
success() && github.event_name == 'push' && github.repository == 'apache/arrow-go' && github.ref_name == 'main'
run: |
ci/scripts/bench.sh --json
- name: Upload results
if: github.event_name == 'push' && github.repository == 'apache/arrow-go' && github.ref_name == 'main'
env:
CONBENCH_URL: https://conbench.ursa.dev
CONBENCH_EMAIL: ${{ secrets.CONBENCH_EMAIL }}
Expand All @@ -61,4 +58,4 @@ jobs:
CONBENCH_MACHINE_INFO_NAME: ${{ matrix.arch }}-debian-12
run: |
python3 -m pip install benchadapt@git+https://github.com/conbench/conbench.git@main#subdirectory=benchadapt/python
python3 ci/scripts/go_bench_adapt.py
python3 ci/scripts/bench_adapt.py
2 changes: 2 additions & 0 deletions ci/scripts/bench_adapt.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
#!/usr/bin/env python3
#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
Expand Down

0 comments on commit 4dc951e

Please sign in to comment.