Skip to content

Remove segmented fetch#33453

Merged
lukasolson merged 3 commits intoelastic:masterfrom
lukasolson:removeSegmentedFetch
Mar 27, 2019
Merged

Remove segmented fetch#33453
lukasolson merged 3 commits intoelastic:masterfrom
lukasolson:removeSegmentedFetch

Conversation

@lukasolson
Copy link
Copy Markdown
Contributor

Replaces #16324.

From the original PR:

Segmented Request was only being used in Discover. It was only actually segmenting requests for old interval based index patterns (and not even for these index patterns in Dashboard). Interval based index patterns are deprecated (cannot be created in 6.0) and will be completely removed in 7.0. The fact that interval based patterns got segmented was an implementation detail of Kibana that is no longer necessary now that ES is smart enough to quickly skip shards that won't return results. The segmented request code creates a huge maintenance burden that prevents us from easily implementing highly requested features like multi-field sorting. This also makes the search fetching behavior more consistent between Discover and Dashboard (which actually relies on the doc table making requests), which will make it easier to extract that logic into a reusable service when we want to rewrite the doc table in React and remove its dependence on Courier.

@lukasolson lukasolson added Feature:Search Querying infrastructure in Kibana Feature:Discover Discover Application Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.0.0 v7.2.0 labels Mar 18, 2019
@lukasolson lukasolson self-assigned this Mar 18, 2019
@lukasolson lukasolson requested a review from Bargs March 18, 2019 21:43
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@lukasolson lukasolson force-pushed the removeSegmentedFetch branch from 756524a to bec23fc Compare March 18, 2019 23:23
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Copy link
Copy Markdown
Contributor

@Bargs Bargs left a comment

Choose a reason for hiding this comment

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

Code and functionality looks good. Inspector and vis are working now, which as far as I can remember were the two last hurdles when I left off. Tested with an old interval based pattern to ensure it still worked and it did. LGTM!

Thanks for taking this across the finish line @lukasolson

@lukasolson lukasolson force-pushed the removeSegmentedFetch branch from bec23fc to 60b16bc Compare March 25, 2019 20:51
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@lukasolson lukasolson force-pushed the removeSegmentedFetch branch from 60b16bc to 0b9d909 Compare March 26, 2019 20:45
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@lukasolson lukasolson merged commit a848840 into elastic:master Mar 27, 2019
lukasolson added a commit to lukasolson/kibana that referenced this pull request Mar 27, 2019
* Remove segmented fetch

* Fix namespace

* Remove unused translations
@lukasolson
Copy link
Copy Markdown
Contributor Author

7.x (7.1.0): 002fc5e

joelgriffith pushed a commit that referenced this pull request Mar 27, 2019
* Remove segmented fetch

* Fix namespace

* Remove unused translations
@timroes timroes added the chore label Apr 3, 2019
@lukasolson lukasolson deleted the removeSegmentedFetch branch December 2, 2019 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Feature:Discover Discover Application Feature:Search Querying infrastructure in Kibana Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.2.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants