fix(arreflect): hasViewableType accepts LARGE_STRING/BINARY; add large+view empty-slice regression test

Author: zeroshade

arrow/array/arreflect/reflect_infer.go

@@ -284,7 +284,7 @@ func applyViewOpts(dt arrow.DataType) arrow.DataType {
 
 func hasViewableType(dt arrow.DataType) bool {
 	switch dt.ID() {
-	case arrow.STRING, arrow.BINARY, arrow.LIST:
+	case arrow.STRING, arrow.BINARY, arrow.LARGE_STRING, arrow.LARGE_BINARY, arrow.LIST:
 		return true
 	case arrow.STRUCT:
 		st := dt.(*arrow.StructType)

arrow/array/arreflect/reflect_infer_test.go

@@ -845,4 +845,10 @@ func TestHasViewableType(t *testing.T) {
 	t.Run("map with string key is true", func(t *testing.T) {
 		assert.True(t, hasViewableType(arrow.MapOf(arrow.BinaryTypes.String, arrow.PrimitiveTypes.Int64)))
 	})
+	t.Run("large_string is true", func(t *testing.T) {
+		assert.True(t, hasViewableType(arrow.BinaryTypes.LargeString))
+	})
+	t.Run("large_binary is true", func(t *testing.T) {
+		assert.True(t, hasViewableType(arrow.BinaryTypes.LargeBinary))
+	})
 }

arrow/array/arreflect/reflect_test.go

@@ -308,6 +308,14 @@ func TestBuildEmptyTyped(t *testing.T) {
 		assert.Equal(t, arrow.STRING_VIEW, llv.Elem().ID())
 	})
 
+	t.Run("large_view_string_empty", func(t *testing.T) {
+		// large applied first: STRING→LARGE_STRING; then view: LARGE_STRING→STRING_VIEW
+		arr, err := buildEmptyTyped(reflect.TypeOf(""), tagOpts{Large: true, View: true}, mem)
+		require.NoError(t, err)
+		defer arr.Release()
+		assert.Equal(t, arrow.STRING_VIEW, arr.DataType().ID())
+	})
+
 	t.Run("dict_with_unsupported_value_type_errors", func(t *testing.T) {
 		_, err := buildEmptyTyped(reflect.TypeOf(time.Time{}), tagOpts{Dict: true}, mem)
 		require.Error(t, err)