Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is RPC GetFileMetadata in Ballista still needed? #963

Closed
yjshen opened this issue Sep 1, 2021 · 3 comments
Closed

Is RPC GetFileMetadata in Ballista still needed? #963

yjshen opened this issue Sep 1, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@yjshen
Copy link
Member

yjshen commented Sep 1, 2021

While working on #932 (comment), I find this GetFileMetadata RPC is not really used, and the intention of the RPC is not clear to me. Do we still need it? Or it's deprecated, and we could safely remove it?

@andygrove, any ideas?

@yjshen yjshen added the enhancement New feature or request label Sep 1, 2021
@andygrove
Copy link
Member

The intent is to update BallistaContext::read_csv / read_parquet methods to retrieve file list and meta data from the scheduler rather than from the local file system so that we can use clients outside of the cluster. For example, I may have a cluster in AWS and I want to run the client on my laptop.

@andygrove
Copy link
Member

See apache/datafusion-ballista#22

@yjshen yjshen closed this as completed Sep 2, 2021
@houqp
Copy link
Member

houqp commented Sep 2, 2021

Ha, good to know. @rdettai FYI since this is related to the new query planning design you are working on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants