-
Notifications
You must be signed in to change notification settings - Fork 884
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
[MD] [Research] Onboard Vis timeseries (TSVB) to use datasource #2153
Comments
@kristenTian Please don't expose internal code amazon link on github |
Thanks Zhongnan. edit the content accordingly |
TSVB
Some actions items for v2.5.0
|
UX refactor discussion , need input from UX teamcc @KrooshalUX @zengyan-amazon Background
Proposal
|
Hi team @huyaboo @zhongnansu would you review the previous research for tsvb to see if there is any idea we could leverage. I'm closing it as @huyaboo is actively working it. |
@seraphjiang Sure, I'll sync with @huyaboo. |
Onboard Vis timeseries (TSVB) to use datasource
Code Ref:
Vis timeseries is also using
getFieldsForWildcard
so data source Id need to be added: /packages/Kibana/blobs/7.10.2/--/src/plugins/vis_type_timeseries/server/lib/search_strategies/strategies/abstract_search_strategy.ts/packages/Kibana/blobs/b03c8bc565d8e58365b9229e2b0bbc41e29731[…]6/--/src/plugins/vis_type_timeseries/server/lib/get_fields.ts
Update to pass datasource in following place:
/packages/Kibana/blobs/b03c8bc565d8e58365b9229e2b0bbc41e2973186/--/src/plugins/vis_type_timeseries/server/lib/get_fields.ts#L48,L76
Front end call: /packages/Kibana/blobs/7.10.2/--/src/plugins/vis_type_timeseries/public/application/lib/fetch_fields.js
->
/packages/Kibana/blobs/f429e08e31b94428cb2efe7b15c1c7dd7ba3ab77/--/src/plugins/vis_type_timeseries/public/application/components/vis_editor.js#L107
The text was updated successfully, but these errors were encountered: