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

Update InternalSearchHit.java #7330

Closed
wants to merge 1 commit into from
Closed

Update InternalSearchHit.java #7330

wants to merge 1 commit into from

Conversation

kuseman
Copy link

@kuseman kuseman commented Aug 19, 2014

Add support for removing meta data from search result

Add support for removing meta data from search result
@clintongormley
Copy link

Hi @kuseman

Thanks for the PR. This wouldn't solve one of the big reasons for#2149 as it still includes the took key in the response, which can vary with every request.

We had decided against supporting this option, but have reopened #2149 after your request to do so. We will discuss again and get back to you.

@kuseman
Copy link
Author

kuseman commented Aug 19, 2014

Ah ok, maybe this PL shouldn't be connected to 2149 but rather a new issue.

@clintongormley
Copy link

Hi @kuseman

We're keen to provide a more generic solution which people can bend to their needs. I've opened this issue about it #7401

thanks for sending the PR, but I'm going to close this issue in favour of #7401

@kuseman
Copy link
Author

kuseman commented Aug 23, 2014

Seems legit, good proposal with transform script, like it.

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.

None yet

2 participants