ysuliu commented on issue #625:
URL: https://github.com/apache/arrow-go/issues/625#issuecomment-3753311718
Hi @zeroshade ,
Thanks for the investigation and the fix. I just tried the fix by applying
the fix locally, I have the further question about `AppendNull()`.
Still based on above testing code, if we add a "Case 3" for appending "Null"
like:
```go
idBuilder.AppendNull()
idBuilder.AppendNull()
idBuilder.AppendNull()
```
Then before your fix, the output is:
```
NULL
NULL
NULL
```
After apply the fix, the output is now:
```
[]
[]
[]
```
This is not as I expected. To my understanding, the result should be
previous all "NULL".
In addition, we can also try a "Case 4" like:
```go
idBuilder.Append([]byte(""))
idBuilder.AppendNull()
idBuilder.Append([]byte("SGVsbG8sIFdvcmxkIQ=="))
```
If we read values as above:
```go
data, _ := col.(*array.Binary)
v := data.Value(rowIdx)
```
What result can we expect ?
Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
