Skip to content

Commit 4dc951e

Browse files
committed
refactor: (GH-#85) Changes in benchmark after review-2
Signed-off-by: Saurabh Kumar Singh <singh1203.ss@gmail.com>
1 parent 1ea6db4 commit 4dc951e

File tree

2 files changed

+8
-9
lines changed

2 files changed

+8
-9
lines changed

.github/workflows/benchmark.yml

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,9 @@ name: Benchmarks
1919
on:
2020
push:
2121
branches: [main]
22-
pull_request:
23-
branches: [main]
2422
paths:
2523
- ".github/workflows/benchmark.yml"
24+
- "ci/scripts/bench_adapt.py"
2625
workflow_dispatch:
2726
permissions:
2827
contents: read
@@ -40,19 +39,17 @@ jobs:
4039
uses: actions/setup-python@v4
4140
with:
4241
python-version: '3.9'
43-
- name: Install dependencies
44-
run: |
45-
apt-get update
46-
apt-get install -y build-essential git python3-pip
4742
- name: Install Go ${{ matrix.go }} for Benchmarks
4843
uses: actions/setup-go@v5
4944
with:
5045
go-version: ${{ matrix.go }}
5146
cache: true
5247
cache-dependency-path: go.sum
5348
- name: Run Benchmarks
54-
if: >-
55-
success() && github.event_name == 'push' && github.repository == 'apache/arrow-go' && github.ref_name == 'main'
49+
run: |
50+
ci/scripts/bench.sh --json
51+
- name: Upload results
52+
if: github.event_name == 'push' && github.repository == 'apache/arrow-go' && github.ref_name == 'main'
5653
env:
5754
CONBENCH_URL: https://conbench.ursa.dev
5855
CONBENCH_EMAIL: ${{ secrets.CONBENCH_EMAIL }}
@@ -61,4 +58,4 @@ jobs:
6158
CONBENCH_MACHINE_INFO_NAME: ${{ matrix.arch }}-debian-12
6259
run: |
6360
python3 -m pip install benchadapt@git+https://github.com/conbench/conbench.git@main#subdirectory=benchadapt/python
64-
python3 ci/scripts/go_bench_adapt.py
61+
python3 ci/scripts/bench_adapt.py

ci/scripts/bench_adapt.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
#!/usr/bin/env python3
2+
#
13
# Licensed to the Apache Software Foundation (ASF) under one
24
# or more contributor license agreements. See the NOTICE file
35
# distributed with this work for additional information

0 commit comments

Comments
 (0)