refactor: (GH-apache#85) Changes in benchmark after review

Signed-off-by: Saurabh Kumar Singh <[email protected]>

Author: singh1203

.github/workflows/benchmark.yml

@@ -21,16 +21,21 @@ on:
     branches: [ main ]
   pull_request:
     branches: [ main ]
+    paths:
+      - ".github/workflows/benchmark.yml"
   workflow_dispatch:
 
+permissions:
+  contents: read
+
 jobs:
   benchmark:
     runs-on: ubuntu-latest
     container: debian:12
 
     strategy:
       matrix:
-        go: ['1.21']
+        go: ['1.22.7']
         arch: ['amd64']
 
     steps:
@@ -47,12 +52,6 @@ jobs:
           apt-get install -y build-essential git python3-pip
           
       - name: Install Go ${{ matrix.go }} for Benchmarks
-        if: >-
-          success() &&
-          matrix.arch == 'amd64' &&
-          github.event_name == 'push' &&
-          github.repository == 'apache/arrow-go' &&
-          github.ref_name == 'main'
         uses: actions/setup-go@v5
         with:
           go-version: ${{ matrix.go }}
@@ -62,7 +61,6 @@ jobs:
       - name: Run Benchmarks
         if: >-
           success() &&
-          matrix.arch == 'amd64' &&
           github.event_name == 'push' &&
           github.repository == 'apache/arrow-go' &&
           github.ref_name == 'main'

ci/scripts/bench.sh

@@ -23,16 +23,6 @@
 
 set -ex
 
-# simplistic semver comparison
-verlte() {
-    [ "$1" = "`echo -e "$1\n$2" | sort -V | head -n1`" ]
-}
-verlt() {
-    [ "$1" = "$2" ] && return 1 || verlte $1 $2
-}
-
-ver=`go env GOVERSION`
-
 source_dir=${1}
 
 export PARQUET_TEST_DATA=${1}/cpp/submodules/parquet-testing/data
@@ -51,4 +41,4 @@ if [[ "$2" = "-json" ]]; then
     cat ${source_dir}/bench_*.dat | gobenchdata --json bench_stats.json
 fi    
 
-rm ${source_dir}/bench_*.dat
+rm ${source_dir}/bench_*.dat

ci/scripts/bench_adapt.py

@@ -124,4 +124,5 @@ def _transform_results(self) -> List[BenchmarkResult]:
 
 if __name__ == "__main__":
     go_adapter = GoAdapter(result_fields_override={"info": {}})
-    go_adapter()
+    go_adapter()
+