apacheGH-38824: [Go] Enable GC checks (apache#38826)

### What changes are included in this PR? Our codebase has many uses of `unsafe.Pointer`. It may be a good idea to add debug checks for these uses in CI builds, especially where data crosses C boundaries. ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * Closes: apache#38824 Authored-by: Antoine Pitrou <[email protected]> Signed-off-by: Antoine Pitrou <[email protected]>
Bit-Quill · Nov 27, 2023 · 1cd22df · 1cd22df
1 parent 6070815
commit 1cd22df
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/ci/scripts/go_build.sh b/ci/scripts/go_build.sh
@@ -21,6 +21,9 @@ set -ex
 
 source_dir=${1}/go
 
+# Need "all=" as per https://github.com/golang/go/issues/42131#issuecomment-713917379
+export GOFLAGS="${GOFLAGS} -gcflags=all=-d=checkptr"
+
 pushd ${source_dir}/arrow
 
 if [[ -n "${ARROW_GO_TESTCGO}" ]]; then

diff --git a/ci/scripts/go_cgo_python_test.sh b/ci/scripts/go_cgo_python_test.sh
@@ -21,6 +21,8 @@ set -ex
 
 source_dir=${1}/go
 
+export GOFLAGS="${GOFLAGS} -gcflags=all=-d=checkptr"
+
 pushd ${source_dir}/arrow/cdata/test
 
 case "$(uname)" in

diff --git a/dev/tasks/tasks.yml b/dev/tasks/tasks.yml
@@ -96,6 +96,9 @@ groups:
   ruby:
     - test-*ruby*
 
+  go:
+    - test*-go-*
+
   vcpkg:
     - test-*vcpkg*
-Original file line number
+Diff line change
@@ Expand Up / @@ -96,6 +96,9 @@ groups: @@
       ruby:
         - test-*ruby*
+      go:
+        - test*-go-*
       vcpkg:
         - test-*vcpkg*
@@ Expand Down @@