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

Set fullMappings #60

Open
P34CH opened this issue May 30, 2017 · 1 comment
Open

Set fullMappings #60

P34CH opened this issue May 30, 2017 · 1 comment

Comments

@P34CH
Copy link

P34CH commented May 30, 2017

Is it correct, that there's currently no possibility to specify the fullMapping property of any Flowpack\ElasticSearch\Domain\Model\Mapping object?

For my current project I need to configure the _all field. That's exactly what the fullMapping property is there for.

In my opinion it would be best to have the option to define the fullMapping as a settings array inside Settings.yaml like this:

Flowpack:
  ElasticSearch:
   indexes:
     [...]
    mapping:
      'person':
        fullMapping:
          _all:
            analyzer: 'nGram_analyzer'
            search_analyzer: 'whitespace_analyzer'

What do you think?

@cwenzelg
Copy link

cwenzelg commented Jan 4, 2018

@dfeyer i think this can be closed, it's there with public function setFullMapping(array $fullMapping), just not as a configuration.

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

No branches or pull requests

2 participants