-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Maps] Remove maps-telemetry saved object as it is no longer in use #69871
Changes from 3 commits
11490df
e2fbf08
f798091
c02e374
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,7 +6,7 @@ | |
import { SavedObjectsType } from 'src/core/server'; | ||
|
||
export const mapsTelemetrySavedObjects: SavedObjectsType = { | ||
name: 'maps', | ||
name: 'maps-telemetry', | ||
hidden: false, | ||
namespaceType: 'agnostic', | ||
mappings: { | ||
|
@@ -37,8 +37,8 @@ export const mapsTelemetrySavedObjects: SavedObjectsType = { | |
avg: { type: 'long' }, | ||
}, | ||
}, | ||
layerTypesCount: { dynamic: 'true', properties: {} }, | ||
emsVectorLayersCount: { dynamic: 'true', properties: {} }, | ||
Comment on lines
-40
to
-41
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Would it be enough if we did There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That might make more sense here as a placeholder since we do want to retain some version of these moving forward but without using dynamic fields. I've added back in the logic supporting construction of these fields and changed their types per your recommendation. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we also need to set There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @rudolf Per the conversation above, I've actually removed this saved object. It appears to be leftover from older logic and is no longer required! |
||
layerTypesCount: { type: 'object' }, | ||
emsVectorLayersCount: { type: 'object' }, | ||
}, | ||
}, | ||
}, | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this constant needs to match the
name
inx-pack/plugins/maps/server/saved_objects/maps_telemetry.ts
:maps-telemetry
. Because we are searching for it in thegetMapsTelemetry
method :)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hold on! Scratch that! Is this related to some other saved objects maps is maintaining?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, looks like you figured this out in the above conversation but this is unrelated. This defines the constant for our saved maps of type
map
.