jorisvandenbossche commented on PR #14343:
URL: https://github.com/apache/arrow/pull/14343#issuecomment-1272923948

   > I have added a warning to the `PlasmaClient` and `ObjectID` class but not 
to `PlasmaBuffer` as it is not meant to be called directly. Happy to change if 
my assumption is wrong.
   
   I am not familiar with plasma myself, but looking at our docs for it 
(https://arrow.apache.org/docs/dev/python/plasma.html): there is also a 
`connect()` function, but I suppose that will call PlasmaClient under the hood. 
   There is also a `plasma_store` command line, which works through 
`pyarrow._plasma_store_entry_point`. That can maybe also print a warning. 
   
   It would also be good to verify in the tests that the warning is being 
raises, and further suppress the warnings (so we don't see them ourselves when 
running pytest). See for example how I did this in 
https://github.com/apache/arrow/pull/8255


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to