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.

Sign up for GitHub

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

Jump to bottom

Optimize QueryOptions to only include entries for non-default value options #2420

Open
wants to merge 57 commits into
base: integration
Choose a base branch
from
Open

Optimize QueryOptions to only include entries for non-default value options #2420

wants to merge 57 commits into from

Conversation

SethSmucker
Copy link
Collaborator

  • Create function evaluateBooleanOption() method in QueryOptions
  • Use evaluateBooleanOption() in the validateOptions() method
  • Add a test in QueryOptionsTest to test evaluateBooleanOption()

resolves #2410

@lbschanno lbschanno marked this pull request as draft July 9, 2024 17:43
@SethSmucker SethSmucker changed the title Add defaulting of non-"false" QueryOption fields to "true" Optimize QueryOptions to only include entries for non-default value options Jul 11, 2024
SethSmucker and others added 19 commits July 12, 2024 11:46
@SethSmucker
Merged integration into TQO
6e0d071
@SethSmucker
Merged integration into TQO
199781a
@SethSmucker @foster33
Update warehouse/query-core/src/main/java/datawave/query/iterator/Que…
bf8b43c 
…ryOptions.java

Co-authored-by: foster33 <[email protected]>
@SethSmucker
Merged integration into TQO
8b8dfd5
@SethSmucker
Formatting
ce8bd48
@SethSmucker
Fixed empty string logical error
1d4366a
@SethSmucker
Fixed empty string logical error
7860443
@lbschanno @SethSmucker
Create initial implementation of way to determin default values
e6d0edb
@lbschanno @SethSmucker
Fix typo
56c976b
@SethSmucker
Temporary commit for bugfix tools
0998782
@SethSmucker
Temporary test commit for my local branch, will be discarded later
d783669
@SethSmucker
testing changes
31dd04f
@SethSmucker
Add default values to createDefaultOptions()
1ea0d9c
@SethSmucker
Add valueTransformer type coverage in addOption()
962cca3
@SethSmucker
Add Object coverage in addOption()
3ef46f8
@SethSmucker @lbschanno
Updated the else condition of addOption()
615190e 
Co-authored-by: Laura Schanno <[email protected]>
@SethSmucker
Removed evaluateBooleanOptions() from validateOptions(), reverting it…
5cacfeb 
… to how it was originally
@SethSmucker
Updated DQP
6d54643
@SethSmucker
Finally rebased
1ddbcbe
@ivakegg
Copy link
Collaborator

ivakegg commented Sep 4, 2024

I had added several comments but then removed them once I figured out what the thought process is here. I think this looks good, but I need to do some integration testing before bringing it in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@foster33 foster33 foster33 left review comments

@lbschanno lbschanno lbschanno requested changes

@ivakegg ivakegg ivakegg approved these changes

@jwomeara jwomeara Awaiting requested review from jwomeara jwomeara is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

@SethSmucker SethSmucker

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Tighter query options
4 participants
@SethSmucker @ivakegg @lbschanno @foster33