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

Better logic on sending mapping update new type introduction #6669

Closed
wants to merge 5 commits into from

Commits on Jul 1, 2014

  1. better logic on sending mapping update new type introduction

    when an indexing request introduces a new mapping, today we rely on the parsing logic to mark it as modified on the "first" parsing phase. This can cause sending of mapping updates to master even when the mapping has been introduced in the create index/put mapping case, and can cause sending mapping updates without needing to.
     This bubbled up in the disabled field data format test, where we explicitly define mappings to not have the update mapping behavior happening, yet it still happens because of the current logic, and because in our test we delay the introduction of any mapping updates randomly, it can get in and override updated ones.
    closes elastic#6669
    kimchy committed Jul 1, 2014
    Copy the full SHA
    3e5e018 View commit details
    Browse the repository at this point in the history
  2. better logging in test

    kimchy committed Jul 1, 2014
    Copy the full SHA
    9e413e3 View commit details
    Browse the repository at this point in the history
  3. Copy the full SHA
    e406dcb View commit details
    Browse the repository at this point in the history

Commits on Jul 2, 2014

  1. Copy the full SHA
    bbbab96 View commit details
    Browse the repository at this point in the history
  2. Copy the full SHA
    ad63dae View commit details
    Browse the repository at this point in the history