hnwyllmm commented on PR #760:
URL: https://github.com/apache/arrow-java/pull/760#issuecomment-2907712247

   > > > Is it possible to add a test?
   > > 
   > > 
   > > I'll try to add a test case in `integration_tests.py`.
   > 
   > Do you need an integration test? IMO, you should be able to export a 
stream, then immediately import it, all within Java.
   
   I add a unittest but I think it can't make sure `TryCopyLastError` works 
correctly, so I suggest it should be removed.
   We can read memory after the error string address in most cases. We usually 
check such problems by sanity tools.
   


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