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

OAK-9310 Filter restriction only performs for the same selector. #342

Closed
wants to merge 1 commit into from

Conversation

mohammadMahdavi
Copy link

This commit solve the problem that I described in OAK-9310. The problem depends on the query plan of execution and in some situations it makes wrong results.

@mohammadMahdavi
Copy link
Author

Hi community,
I want to know why no one checks my PR? Is there any guidance that I should create PR based on it for approval?
@anchela @rishabhdaim @joerghoh

@mreutegg
Copy link
Contributor

@mohammadMahdavi can you please add an Apache license header to your test class FullTextSearchImplTest? It currently fails the build. It would also be good if the PR does not contain whitespace changes.

@thomasmueller, WDYT about the test and proposed change?

@mohammadMahdavi
Copy link
Author

@mreutegg I did the changes.

Copy link

This PR is stale because it has been open 24 months with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the Stale label Jul 19, 2024
@github-actions github-actions bot closed this Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants