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 <antoine@python.org> Signed-off-by: Antoine Pitrou <antoine@python.org>
dgreiss · Feb 17, 2024 · 3705f8b · 3705f8b
1 parent c95d802
commit 3705f8b
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*