Tom-Newton commented on issue #40025:
URL: https://github.com/apache/arrow/issues/40025#issuecomment-1939180865

   >>Assuming https://github.com/apache/arrow/pull/40021 is completed first, 
then this ticket should include enabling the python side tests for move. I will 
add TODO(GH-40025) comments where relevant.
   
   > If you can pass the tests on a real HNS storage account, it should be the 
same test code running against a different environment. The semantics should be 
the same. I don't plan to add new test cases in azurefs_test.cc for Move() on 
flat namespace accounts because the test code is supposed to run the same way 
on these different environments.
   
   That all makes sense but I think we are talking cross purposes. I'm not 
proposing we add new tests. The python tests are generic across all filesystems 
and https://github.com/apache/arrow/pull/40021 just enables them to run against 
azurite. However it currently it disables a couple of tests because move is not 
supported on azurite. I don't think its worth running the python tests against 
real blob storage. 


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