alamb commented on PR #3824:
URL: https://github.com/apache/arrow-rs/pull/3824#issuecomment-1464735343

   > > My experience of target-specific feature selection has been that they 
cause very strange and hard to debug problems, the ahash getrandom nonsense is 
a frequent source of pain.
   
   > I wonder if we could instead add a default-disabled top-level feature, 
maybe append_local that gates this functionality? This would also help with 
flagging the functionality as not part of the standard object store contract.
   
   I am not sure about the WASM usecase in general for object_store (as in I 
don't really understand why it is an  an important target to support for 
object_store at all). If we are looking for simplifications maybe we can remove 
that feature support instead of making a new feature flag 🤔 
   
   
   I agree with @ozankabak that making a new feature flag for append seems 
overly complicated. I think this would be fine to merge as is. 
   
   What do you think @tustvold ?


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