wgtmac commented on PR #35825:
URL: https://github.com/apache/arrow/pull/35825#issuecomment-1599077161

   > @wgtmac I have just added a string test, can you confirm it's the type of 
test you want? 
[7f09a16](https://github.com/apache/arrow/commit/7f09a160730425d933a433104f09782a50633615).
   > 
   > If so, I'll try to create a similar one for the list type.
   
   IMO, we can ignore the test case for `list<string>` if there is a test for 
`map<string,xx>`. They should be same to the string column writer. IIRC, single 
primitive string type does not have any issues while reading, can you confirm? 
If that is true, we can also ignore single string type in the test file.


-- 
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]

Reply via email to