[GitHub] [arrow] sbinet commented on a change in pull request #7823: ARROW-9548: [Go] Test output files are not removed correctly

2020-08-07 Thread GitBox
sbinet commented on a change in pull request #7823: URL: https://github.com/apache/arrow/pull/7823#discussion_r467016096 ## File path: go/arrow/internal/arrjson/arrjson_test.go ## @@ -36,12 +42,11 @@ func TestReadWrite(t *testing.T) { mem := memory.NewC

[GitHub] [arrow] sbinet commented on a change in pull request #7823: ARROW-9548: [Go] Test output files are not removed correctly

2020-08-07 Thread GitBox
sbinet commented on a change in pull request #7823: URL: https://github.com/apache/arrow/pull/7823#discussion_r466958913 ## File path: go/arrow/arrio/arrio_test.go ## @@ -94,19 +100,17 @@ func TestCopy(t *testing.T) { mem

[GitHub] [arrow] sbinet commented on a change in pull request #7823: ARROW-9548: [Go] Test output files are not removed correctly

2020-08-04 Thread GitBox
sbinet commented on a change in pull request #7823: URL: https://github.com/apache/arrow/pull/7823#discussion_r465023873 ## File path: go/arrow/ipc/cmd/arrow-json-integration-test/main.go ## @@ -158,16 +148,6 @@ func cnvToARROW(arrowName, jsonName string, verbose bool) error {