BlakeOrth opened a new pull request, #17953:
URL: https://github.com/apache/datafusion/pull/17953
## Which issue does this PR close?
This does not fully close, but is an incremental building block component
to:
- https://github.com/apache/datafusion/issues/17207
Functionally this code currently does nothing, but it's an important
component that will allow access to future instrumented object stores. The full
context of how this code is likely to progress can be seen in the POC for this
effort:
- https://github.com/apache/datafusion/pull/17266
(note: If we _need_ an individual issue to close for process reasons I am
happy to make one and associate it with #17207 as part of that larger effort)
## Rationale for this change
The full implementation for #17207 is likely too large for a single PR, so
to better account for review bandwidth we would like to break the
implementation into smaller pieces for easier review. This happens to be about
the smallest and least intrusive atomic unit I could think of based on the POC
work.
## What changes are included in this PR?
- Adds a new Object Store Registry wrapper to datafusion-cli to support
interacting with instrumented object store instances
- Introduces a new `instrumented` sub-module to the `object_storage` module
to house all future instrumented object store code
- Adds basic tests for the new registry wrapper
## Are these changes tested?
Yes, new tests have been introduced and all existing tests are passing
## Are there any user-facing changes?
No
cc: @alamb
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
