Skip to content

Commit

Permalink
[savedObjects] change the number mappings to be 'long'
Browse files Browse the repository at this point in the history
Since JavaScript numbers can't actually be integers we don't really care about
the difference between int and long, but since the long is the default type it
makes things more frictionless
  • Loading branch information
Spencer Alger committed May 21, 2015
1 parent 05baf24 commit 1b9a646
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions src/kibana/plugins/dashboard/services/_saved_dashboard.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ define(function (require) {
// if type:dashboard has no mapping, we push this mapping into ES
mapping: {
title: 'string',
hits: 'integer',
hits: 'long',
description: 'string',
panelsJSON: 'string',
version: 'integer',
version: 'long',
timeRestore: 'boolean',
timeTo: 'string',
timeFrom: 'string'
Expand Down
6 changes: 3 additions & 3 deletions src/kibana/plugins/discover/saved_searches/_saved_search.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@ define(function (require) {
mapping: {
title: 'string',
description: 'string',
hits: 'integer',
hits: 'long',
columns: 'string',
sort: 'string',
version: 'integer'
version: 'long'
},

defaults: {
Expand All @@ -42,4 +42,4 @@ define(function (require) {

return SavedSearch;
});
});
});
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ define(function (require) {
visState: 'json',
description: 'string',
savedSearchId: 'string',
version: 'integer'
version: 'long'
},

defaults: {
Expand Down

0 comments on commit 1b9a646

Please sign in to comment.