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

GetFieldMappings API will not return field mapping if the index is not hosted on the node executing it #5177

Closed
bleskes opened this issue Feb 19, 2014 · 1 comment

Comments

Projects
None yet
4 participants
@bleskes
Copy link
Member

commented Feb 19, 2014

The current implementation of the GetFieldMapping API uses information from the index service which is only available on a node if that node actively hosts a shard from that index. If the information is missing the call will act as if the type/field was not found and will not return information for it.

During a rolling upgrade from <= 0.90.11 or 1.0.0 the get field mapping api might fail. This has to do with the way this issue has been fixed. The way how internally the request got handled has changed in order completely to fix it properly at the cost that during a rolling upgrade this api may fail.

@darkwarriors

This comment has been minimized.

Copy link
Contributor

commented Feb 20, 2014

any news on this bug? we are facing the same error by upgrading ES from .90 to 1.0.

Regards

@martijnvg martijnvg assigned martijnvg and unassigned bleskes Feb 21, 2014

martijnvg added a commit to martijnvg/elasticsearch that referenced this issue Feb 21, 2014

Change GetFieldMapping API to broadcast requests to nodes hosting the…
… relevant indices.

This is due to the fact that have to have mappers in order to return the response.

Closes elastic#5177

@javanna javanna added v1.0.2 and removed v1.0.2 labels Feb 24, 2014

rashidkpc pushed a commit to rashidkpc/kibana that referenced this issue Feb 24, 2014

bleskes added a commit that referenced this issue Feb 24, 2014

Change GetFieldMapping API to broadcast requests to nodes hosting the…
… relevant indices.

This is due to the fact that have to have mappers in order to return the response.

Closes #5177

@bleskes bleskes closed this in 3e10810 Feb 24, 2014

bleskes added a commit that referenced this issue Feb 24, 2014

Change GetFieldMapping API to broadcast requests to nodes hosting the…
… relevant indices.

This is due to the fact that have to have mappers in order to return the response.

Closes #5177

bleskes added a commit that referenced this issue Feb 24, 2014

Change GetFieldMapping API to broadcast requests to nodes hosting the…
… relevant indices.

This is due to the fact that have to have mappers in order to return the response.

Closes #5177

martijnvg added a commit that referenced this issue Feb 24, 2014

gabegorelick added a commit to gabegorelick/fracas that referenced this issue Sep 2, 2014

Squashed 'kibana/' changes from fb7e4b6..5bc265e
5bc265e Merge pull request #851 from jdve/fix-histogram-searching
072591b Merge pull request #985 from rashidkpc/master
d513abd URL encode id, closes #949
aef27d4 Remove unsed attrs
5a0dabd Allow multiple bettermaps per page. Closes #984, closes #952
e00654e Add another uniq for folks with poorly configured index patterns
7292afb Fix for height in IE, closes #882
1a579be Merge pull request #866 from delfuego/865
02a8987 histogram: on error search remaining indices
7359357 remove debug line
ec86990 Merge remote-tracking branch 'oneironautics/master' into hitsTips
423460d Fix for map height in IE. Closes #794
7b6d6f0 Merge pull request #786 from shyam0586/master
9f4a5d8 Add pre-wrap to table cells. Closes #778.
a0f20aa Tweak table css. Closes #605. Closes #207. Wrap details view nicely
828e36c Limit the length of query tooltips. Closes #458
168a9e3 Added a favicon. Closes #141
97983fa Fixed missing zoom links, fix inspect icon
6cb9d1f Merge pull request #981 from rashidkpc/master
16f34d5 Merge remote-tracking branch 'kuwerty/interval_to_ms_fix' into subsecond_fix
15f6b8b Merge pull request #980 from rashidkpc/master
75b458e Revert from _mapping/field due to elastic/elasticsearch#5177 . Updated fields service and proxies
b73b11f Fixes histogram plot disappears when time range is too small (issue #733)
2c2a0aa Merge pull request #965 from ccowan/master
e820909 Closing the astrick security hole
cade65d Merge pull request #961 from FestivalBobcats/master
7f1d9ea Merge pull request #955 from bleskes/fix/rev_replacing
a275ed2 Merge pull request #908 from ccowan/master
ff98c3e spelling correction for "uncluttered" in sparklines/module.js
16ab692 The build @rev@ replacement task was breaking comment lines starting with // and containing @rev@
84ed406 Merge pull request #953 from vaionicle/master
02dd5e0 fixing missing span element
05c64fa Don't display field value in tooltip. Closes #941
8512132 Merge pull request #871 from spenceralger/lodash_tests
968f126 Merge pull request #937 from rashidkpc/master
46ec653 Move formating to filter. Closes #925
797bdf0 Remove add button when row is not editable. Fix expand button not showing when row is not editable. Closes #919
88f85fc Merge pull request #936 from rashidkpc/master
b67866b Merge remote-tracking branch 'driskell/bugs/safari7_export_schema_fix' into blob
d55ad64 Filter builder and facets should use same value. Closes #845
04eb85f Use query string instead of field filter. Closes #931
d6f7aa1 Merge pull request #930 from rashidkpc/master
14458a8 Fix potential xss in bootstrap typeahead. Thanks @modlearning
37047ee Fixing proxied spelling
364119f Adding the kibana-int-user code and cleaning things up
a7e6d46 Renaming the directory
b630316 Removing old script
ba3ddf6 Renaming the generate script and fixing a typo in the comments
8ea8c35 Initial import of user auth via filtered aliases
fd23205 Merge pull request #907 from martinseener/master
a6b414b fixed naming of _mapping/field where its field not fields
c312e05 Merge pull request #906 from rashidkpc/master
a888165 reenable regex query
fabdaf8 Updated example proxies
fe65144 Update error message for mapping end point
2b9ad2f Merge branch '10mappings'
1557d4b Merge pull request #905 from rashidkpc/master
0ade329 Remove debug
496585c Support for multiple index patterns, closes #22
2a81e98 Load blob.js along with filesaver.js to fix Export Schema option in Safari 7
94a6236 Merge pull request #898 from rashidkpc/10mappings
22b061d Fix query help link
70d287a Remove debug
607c0ba Enforce minimum version
e2793de Merge pull request #889 from pharaujo/master
d16ab27 Removed version filter for query types, replaced with pre-filtered. Switched to _mapping/fields/ for mapping discovery, bumped min Kibana version to 0.90.9
fc3245f Reverted change to vendor code
56e16ad Fixed 'seperated' spelling error
acb0714 Revert removal of subtle border radius on inputs
9d6573e merged spenceralger/kibana#minor_style_adjustments
ac1bfea Merge pull request #874 from spenceralger/row_text
7edc2c7 Merge pull request #870 from spenceralger/localstorage_store
2c19a45 Merge pull request #869 from spenceralger/lodash
07f7bdf set the row text left aligned, which was already happening as long as the text was more than 30px wide
4fc0bfd Removed the tests from the build
de5118d added tests for lodash mixins
3d01a71 general cleanup after last commmit
24b2796 Adds a store factory
0e134e3 switched out underscore with lodash
09147f2 minor style adjustments to remove border-radius, replace the margin, and include the border on the timepicker
b91227b Merge remote-tracking branch 'upstream/master'
0e9ae70 Closes #861
a92a7fc fix typo in table panel config page
4b8f34e Merge pull request #863 from spenceralger/refresh_icon_ie_fix
35ef4cc made the refresh icon a child of the refresh link, it is now styled correcting in IE
05a7a33 Use -moz-box-sizing for firefox kibana-container
3c8d073 Improve annotation tooltip on light theme
acd96e1 Get rid of the reverse
b8fff57 Re-fix overly broad cluster, events need to be sorted correctly
1f4d399 Fixed overly broad event clustering. Re: #854
93b6c69 Merge remote-tracking branch 'marutib/master' into dashTitle
398ac24 histogram: on error search remaining indices
234094d Resolve ids from list at initialization
e127506 Added license information for vendor packages
e67a8f8 make panels 100% width on mobile
f8e34a6 Improve button positioning
688f7f8 Improve droppable spaces when dragging
816b1cc Revert max span to 12
8621a44 Align row buttons with panels
c12ccaf Improve visibility of available drag spaces
a8fce6b Add subtle border to panels, improve dragging representation
661093a Fix column panel, closes #832
74b771e Fixed resize events
7edc20a Extended save/cancel to row and dashboard editors. Allow rows to overflow gracefully. Normalized spacing between panels
98640fd Add the content of the panel to a panel-content container
b4da2f7 Add git commit to built versions
809174b cleanup comment
5866030 Fixed conflicting setSelection in flot.events
a0c638a Enable clustering in annotations
81f8be0 Tooltip should say 1s if scaleSeconds is set. Fix for route change when clicking on auto-refresh submenu
3b9691a Add number formatting to histogram tooltip until we have a global number formatter
0e68f9d Fixed min and max modes of histogram
aa95ce6 Merge remote-tracking branch 'upstream/master'
4827c7f flipped the chevrons around and lower cased the sort field
c751890 Merge remote-tracking branch 'remip2/master' into stats_sort
5505e37 Merge pull request #821 from martinseener/master
c11837b Correctly compute mean/min/max on histogram intervals larger than the index interval
3bfa407 Fix linting
d836a41 Merge remote-tracking branch 'bleskes/double_refresh' into boaz
f399f5b refresh was called twice with every timer expiry
587d81e added new *./_aliases proxy match for nginx and apache sample config files; fixes #811
b796a1b Merge remote-tracking branch 'bleskes/y_format_sensitive_to_tick_size' into boaz
9dedafc Merge remote-tracking branch 'bleskes/hist_search_type_count' into boaz
9fb504f Add support for version suffixes (like RC1 & Beta) to esVersion parsing & comparison
bd60cf0 Always show the refresh button
684c587 Histrograms should use the count search type when they only rely on facets.
b29a983 Check if time_series bucket already exists, if it does, add to it. Closes #816
ba8f988 Fixed spacing of derivative text, shortened transition delay for row buttons
745b7ad Make histogram y-axis formatting aware of the tick size when formatting
76568ab Merge branch 'master' of github.com:oneironautics/kibana
a93c782 762: add series label to hits panel on hover.
8f4339a fix header alignment
e40ac95 add sortable headers to the stats panel result table
f7b0f63 Show load and save options by default, even when on an invalid dashboard
629705e Merge remote-tracking branch 'bleskes/selective_aliases' into selective_aliases
48cff63 Fix for persistent tooltips on histogram markers
17d8e1a add an option to sort the stat panel results by label or value
4ad4627 Improved index resolving efficiency
455b925 Detailed steps and link to grunt build process
ad3ed16 Run the grunt build process added in FAQ
8c95e3f 762: add series label to hits panel on hover.
e52e72d Closes #671 Changes to set the browser title to the title of the dashboard.

git-subtree-dir: kibana
git-subtree-split: 5bc265ee6a141b5b2260b08a073996a7c75969e1

mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015

Change GetFieldMapping API to broadcast requests to nodes hosting the…
… relevant indices.

This is due to the fact that have to have mappers in order to return the response.

Closes elastic#5177

mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015

mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015

Change GetFieldMapping API to broadcast requests to nodes hosting the…
… relevant indices.

This is due to the fact that have to have mappers in order to return the response.

Closes elastic#5177
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.