fix: Disallow null character in strings per utf8 spec (#797)

This should solve bunch of issues that we have with NullValues and StripNulls logic propagated to all our destination plugins. Apparently [Valid](apache/arrow#35161 (comment)) utf8 string can't have null values in the middle of the string so we should just strip those in our type system (it can have at the end or the start but I suggest actually stripping those down as well as it's really unnecessary as far as I know and it exist prob for some archaic reason like sending a string over the wire without any additional encoding/protocol, which we have via arrow).

Once we move sources to arrow we will add the validation/strip on the source side.

Author: yevgenypats

schema/arrow.go

@@ -2,6 +2,7 @@ package schema
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/goccy/go-json"
 
@@ -270,7 +271,9 @@ func CQTypesToRecord(mem memory.Allocator, c []CQTypes, arrowSchema *arrow.Schem
 				}
 			case TypeString:
 				if c[j][i].(*Text).Status == Present {
-					bldr.Field(i).(*array.StringBuilder).Append(c[j][i].(*Text).Str)
+					// In the new type system we wont allow null string as they are not valid utf-8
+					// https://github.com/apache/arrow/pull/35161#discussion_r1170516104
+					bldr.Field(i).(*array.StringBuilder).Append(strings.ReplaceAll(c[j][i].(*Text).Str, "\x00", ""))
 				} else {
 					bldr.Field(i).(*array.StringBuilder).AppendNull()
 				}
@@ -285,7 +288,7 @@ func CQTypesToRecord(mem memory.Allocator, c []CQTypes, arrowSchema *arrow.Schem
 					listBldr := bldr.Field(i).(*array.ListBuilder)
 					listBldr.Append(true)
 					for _, str := range c[j][i].(*TextArray).Elements {
-						listBldr.ValueBuilder().(*array.StringBuilder).Append(str.Str)
+						listBldr.ValueBuilder().(*array.StringBuilder).Append(strings.ReplaceAll(str.Str, "\x00", ""))
 					}
 				} else {
 					bldr.Field(i).(*array.ListBuilder).AppendNull()