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

Ignore empty fields indexer #1019

Merged
merged 4 commits into from
Dec 20, 2022
Merged

Ignore empty fields indexer #1019

merged 4 commits into from
Dec 20, 2022

Conversation

jnioche
Copy link
Contributor

@jnioche jnioche commented Dec 14, 2022

The indexers currently generate fields where the value can be empty. This PR introduces a new configuration

indexer.ignore.empty.fields with a default value of false.

If set to true, the indexing classes will not generate a field if its value is empty or null.

Signed-off-by: Julien Nioche <julien@digitalpebble.com>
Signed-off-by: Julien Nioche <julien@digitalpebble.com>
@jnioche jnioche added this to the 2.7 milestone Dec 14, 2022
@jnioche jnioche assigned jnioche and unassigned jnioche Dec 15, 2022
@jnioche jnioche self-assigned this Dec 15, 2022
@jnioche
Copy link
Contributor Author

jnioche commented Dec 15, 2022

@rzo1 @sebastian-nagel do you think the default value should be true?

Copy link
Contributor

@rzo1 rzo1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be consistent with current behaviour if it remains false. However, I don't have a strong opinion here.

Signed-off-by: Julien Nioche <julien@digitalpebble.com>
@jnioche jnioche merged commit 1025365 into master Dec 20, 2022
@jnioche jnioche deleted the ignoreEmptyFieldsIndexer branch December 20, 2022 08:48
@jnioche
Copy link
Contributor Author

jnioche commented Dec 20, 2022

thanks a lot @rzo1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants