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

[Maps][File upload] Set complete on index pattern creation #44423

Merged
merged 7 commits into from
Sep 10, 2019

Conversation

kindsun
Copy link
Contributor

@kindsun kindsun commented Aug 29, 2019

Fixes issue where final status was blank on completion and also inaccurate in the code. Indexing process should be complete on creation of index pattern rather than index itself:

image

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis

@nik9000
Copy link
Member

nik9000 commented Aug 29, 2019

@elasticmachine run elasticsearch-ci/docs

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@kindsun kindsun requested a review from nickpeihl August 30, 2019 15:02
Copy link
Member

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something is undefined.

Screen Shot 2019-09-03 at 11 18 50 AM

Also, I'm stuck on that screen and getting the following error in the browser console.

Uncaught (in promise) TypeError: Cannot read property 'failures' of undefined
    at eval (geojson_file_source.js:62)
    at _callee$ (json_upload_and_parse.js:141)
    at tryCatch (runtime.js:45)
    at Generator.invoke [as _invoke] (runtime.js:271)
    at Generator.prototype.<computed> [as next] (runtime.js:97)
    at asyncGeneratorStep (json_upload_and_parse.js:19)
    at _next (json_upload_and_parse.js:19)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@kindsun
Copy link
Contributor Author

kindsun commented Sep 5, 2019

@nickpeihl Can you verify this behavior by giving it another shot? I tried multiple files on both Firefox and Chrome and was unable to replicate this behavior. Once you've verified it, if possible, could you upload the file?

@elasticmachine
Copy link
Contributor

💔 Build Failed

@nickpeihl nickpeihl self-requested a review September 10, 2019 01:21
Copy link
Member

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm if ci passes.

@kindsun
Copy link
Contributor Author

kindsun commented Sep 10, 2019

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@kindsun kindsun merged commit 8114b53 into elastic:master Sep 10, 2019
kindsun pushed a commit to kindsun/kibana that referenced this pull request Sep 10, 2019
…4423)

* Set complete on index pattern creation

* Set complete on error as well. Pass resp to callback function on timeout error
kindsun pushed a commit to kindsun/kibana that referenced this pull request Sep 10, 2019
…4423)

* Set complete on index pattern creation

* Set complete on error as well. Pass resp to callback function on timeout error
kindsun pushed a commit that referenced this pull request Sep 10, 2019
…45228)

* Set complete on index pattern creation

* Set complete on error as well. Pass resp to callback function on timeout error
kindsun pushed a commit that referenced this pull request Sep 10, 2019
…45229)

* Set complete on index pattern creation

* Set complete on error as well. Pass resp to callback function on timeout error
jloleysens added a commit to jloleysens/kibana that referenced this pull request Sep 10, 2019
…-to-np-ready

* 'master' of github.com:elastic/kibana: (138 commits)
  [Canvas] i18n work on workpad header (and a few header CTAs) and convert to typescript (elastic#44943)
  update close/delete system index modals (elastic#45037)
  TS return type of createIndexPatternSelect (elastic#45107)
  [ML] Fix focus chart updating. (elastic#45146)
  [ML] Data frame transform: Fix progress in wizard create step. (elastic#45116)
  [Graph] Re-enable functional test (elastic#44683)
  [SIEM] unique table id for each top talkers table (elastic#45014)
  [SIEM] ip details heading draggable (elastic#45179)
  [Maps][File upload] Set complete on index pattern creation (elastic#44423)
  [Maps] unmount map embeddable component on destroy (elastic#45183)
  [SIEM] Adds error toasts to MapEmbeddable component (elastic#45088)
  fix redirect to maintain search query string (elastic#45184)
  [APM] One-line trace summary (elastic#44842)
  [Infra UI] Display non-metric details on Node Detail page (elastic#43551)
  [Maps][File upload] Removing bbox from parsed file pending upstream lib fix (elastic#45194)
  [Logs UI] Improve live streaming behavior when scrolling (elastic#44923)
  [APM] Fix indefinite loading state in agent settings for unauthorized user roles (elastic#44970)
  [Reporting] Rewrite addForceNowQuerystring to getFullUrls (elastic#44851)
  [Reporting/ESQueue] Improve logging of doc-update events (elastic#45077)
  [Reporting] Make screenshot capture less noisy by default (elastic#45185)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants