GH-41887: [Go] Run linter via pre-commit

.gitignore

@@ -102,4 +102,9 @@ __debug_bin
 .envrc
 
 # Develocity
+.mvn/.gradle-enterprise/
 .mvn/.develocity/
+
+# rat
+filtered_rat.txt
+rat.txt

.pre-commit-config.yaml

@@ -141,3 +141,8 @@ repos:
           '--disable',
           'dangling-hyphen,line-too-long',
         ]
+  - repo: https://github.com/dnephin/pre-commit-golang
+    rev: v0.5.1
+    hooks:
+      - id: go-fmt
+      - id: go-imports

go/arrow/array/bufferbuilder_numeric_test.go

@@ -20,8 +20,8 @@ import (
 	"testing"
 	"unsafe"
 
-	"github.com/apache/arrow/go/v17/arrow/memory"
 	"github.com/apache/arrow/go/v17/arrow/endian"
+	"github.com/apache/arrow/go/v17/arrow/memory"
 	"github.com/stretchr/testify/assert"
 )
 

go/arrow/array/numeric_test.go

@@ -16,7 +16,7 @@
 
 package array_test
 
-import (	
+import (
 	"math"
 	"reflect"
 	"testing"
@@ -144,7 +144,7 @@ func TestFloat16MarshalJSON(t *testing.T) {
 
 	bldr := array.NewFloat16Builder(pool)
 	defer bldr.Release()
-	
+
 	jsonstr := `[0, 1, 2, 3, "NaN", "NaN", 4, 5, "+Inf", "-Inf"]`
 
 	bldr.Append(float16.New(0))
@@ -158,7 +158,6 @@ func TestFloat16MarshalJSON(t *testing.T) {
 	bldr.Append(float16.Inf())
 	bldr.Append(float16.Inf().Negate())
 
-
 	expected := bldr.NewFloat16Array()
 	defer expected.Release()
 	expected_json, err := expected.MarshalJSON()
@@ -172,7 +171,7 @@ func TestFloat32MarshalJSON(t *testing.T) {
 
 	bldr := array.NewFloat32Builder(pool)
 	defer bldr.Release()
-	
+
 	jsonstr := `[0, 1, "+Inf", 2, 3, "NaN", "NaN", 4, 5, "-Inf"]`
 
 	bldr.Append(0)
@@ -186,10 +185,9 @@ func TestFloat32MarshalJSON(t *testing.T) {
 	bldr.Append(5)
 	bldr.Append(float32(math.Inf(-1)))
 
-
 	expected := bldr.NewFloat32Array()
 	defer expected.Release()
-	
+
 	expected_json, err := expected.MarshalJSON()
 	assert.NoError(t, err)
 
@@ -223,7 +221,7 @@ func TestFloat64MarshalJSON(t *testing.T) {
 	assert.NoError(t, err)
 
 	assert.JSONEq(t, jsonstr, string(expected_json))
-	
+
 }
 
 func TestUnmarshalSpecialFloat(t *testing.T) {

go/arrow/array/record_test.go

@@ -94,7 +94,7 @@ func TestRecord(t *testing.T) {
 	if _, err := rec.SetColumn(0, col2_1); err == nil {
 		t.Fatalf("expected an error")
 	}
-	newRec, err := rec.SetColumn(1, col2_1);
+	newRec, err := rec.SetColumn(1, col2_1)
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}

go/arrow/float16/float16.go

@@ -175,7 +175,7 @@ func (n Num) Signbit() bool { return (n.bits & 0x8000) != 0 }
 
 func (n Num) IsNaN() bool { return (n.bits & 0x7fff) > 0x7c00 }
 
-func (n Num) IsInf() bool {return (n.bits & 0x7c00) == 0x7c00 }
+func (n Num) IsInf() bool { return (n.bits & 0x7c00) == 0x7c00 }
 
 func (n Num) IsZero() bool { return (n.bits & 0x7fff) == 0 }
 

go/arrow/gen-flatbuffers.go

@@ -14,6 +14,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+//go:build ignore
 // +build ignore
 
 package main

go/arrow/internal/debug/assert_off.go

@@ -14,6 +14,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+//go:build !assert
 // +build !assert
 
 package debug

go/arrow/internal/debug/assert_on.go

@@ -14,6 +14,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+//go:build assert
 // +build assert
 
 package debug

go/arrow/internal/debug/doc.go

@@ -17,14 +17,12 @@
 /*
 Package debug provides APIs for conditional runtime assertions and debug logging.
 
-
-Using Assert
+# Using Assert
 
 To enable runtime assertions, build with the assert tag. When the assert tag is omitted,
 the code for the assertion will be omitted from the binary.
 
-
-Using Log
+# Using Log
 
 To enable runtime debug logs, build with the debug tag. When the debug tag is omitted,
 the code for logging will be omitted from the binary.

go/arrow/internal/debug/log_off.go

@@ -14,6 +14,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+//go:build !debug
 // +build !debug
 
 package debug

go/arrow/internal/debug/log_on.go

@@ -14,6 +14,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+//go:build debug
 // +build debug
 
 package debug

go/arrow/internal/debug/util.go

@@ -14,6 +14,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+//go:build debug || assert
 // +build debug assert
 
 package debug