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

FedX does not correctly reduce federation members, when the query contains a FROM clause #5030

Closed
aschwarte10 opened this issue Jun 14, 2024 · 0 comments · Fixed by #5031
Closed
Assignees
Labels
🐞 bug issue is a bug 📦 fedx fedx: optimized federated query support
Milestone

Comments

@aschwarte10
Copy link
Contributor

Current Behavior

FedX supports interpret the Dataset being defined on the query: if it is of type FedXDataset, the federation is reduced to the members defined by FedXDataset#getEndpoints().

This is a nice means to evaluate a query on a subset of the federation and is generally working fine.

When the query contains a FROM clause, however, the externally passed Dataset is wrapped into a FallbackDataset (which internally keeps the FedXDataset as primary one.

Expected Behavior

The federation engine correctly respects an externally passed FedXDatasetto reduce the federation

Steps To Reproduce

See test case in the provided PR

Version

4.8.12

Are you interested in contributing a solution yourself?

Yes

Anything else?

Would like to see this merged still as part of 5.0

@aschwarte10 aschwarte10 added the 🐞 bug issue is a bug label Jun 14, 2024
@aschwarte10 aschwarte10 added this to the 5.0.0 milestone Jun 14, 2024
@aschwarte10 aschwarte10 self-assigned this Jun 14, 2024
@aschwarte10 aschwarte10 added the 📦 fedx fedx: optimized federated query support label Jun 14, 2024
aschwarte10 added a commit that referenced this issue Jun 14, 2024
FedX supports interpret the Dataset being defined on the query: if it is
of type FedXDataset, the federation is reduced to the members defined by
FedXDataset#getEndpoints().

This is a nice means to evaluate a query on a subset of the federation
and is generally working fine.

When the query contains a FROM clause, however, the externally passed
Dataset is wrapped into a FallbackDataset (which internally keeps the
FedXDataset as primary one.

This change now inspects the FallbackDataset and unwraps it to reduce
the members. Note that for execution still the "FallbackDataset" is
used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug issue is a bug 📦 fedx fedx: optimized federated query support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant