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

make some datasource listing helper functions public? #10449

Open
samuelcolvin opened this issue May 10, 2024 · 1 comment
Open

make some datasource listing helper functions public? #10449

samuelcolvin opened this issue May 10, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@samuelcolvin
Copy link
Contributor

Is your feature request related to a problem or challenge?

I'm trying to implement a custom variant of ListingTable, and I'm running into an issue that the following three helper functions are currently not public:

pub async fn pruned_partition_list<'a>(

pub fn expr_applicable_for_cols(col_names: &[String], expr: &Expr) -> bool {

That alone prevents me from implementing my own ListingTable with an otherwise reasonably level of complexity.

Describe the solution you'd like

I'd like either:

Describe alternatives you've considered

I'd rather not copy and paste the whole of /datafusion/core/src/datasource/listing/helpers.rs, but that seems to be the only way around this.

Additional context

No response

@samuelcolvin samuelcolvin added the enhancement New feature or request label May 10, 2024
@alamb
Copy link
Contributor

alamb commented May 12, 2024

I think making the functions public would be fine.

In general I think it would be super valuable to pull the ListingTable code out of the core crate (into datafusion-listing-table for example), as that would help make sure the APIs that are needed to use it are all publically exposed / documented. LIstingTable as implemented today is non trivial to reuse

However I think it would take concerted effort to pull LIstingTable out

Here is a related discussion #8345 where others have a similar idea

samuelcolvin added a commit to samuelcolvin/datafusion that referenced this issue May 19, 2024
samuelcolvin added a commit to samuelcolvin/datafusion that referenced this issue May 22, 2024
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

No branches or pull requests

2 participants