Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[android] Convert GeoJSON features to tiles in background #15871

Merged
merged 3 commits into from
Oct 31, 2019

Conversation

pozdnyakov
Copy link
Contributor

@pozdnyakov pozdnyakov commented Oct 30, 2019

Composing tiles from the GeoJSON features is an expensive operation that might block UI thread on updating the GeoJsonSource with the new data.

This change moves tile composing to the background thread and thus unblocks the UI thread.

Fixes: https://github.com/mapbox/mapbox-gl-native-team/issues/87
Tag: #8484

@pozdnyakov pozdnyakov added performance Speed, stability, CPU usage, memory usage, or power usage Android Mapbox Maps SDK for Android labels Oct 30, 2019
@pozdnyakov pozdnyakov self-assigned this Oct 30, 2019
@pozdnyakov pozdnyakov added the needs changelog Indicates PR needs a changelog entry prior to merging. label Oct 30, 2019
@pozdnyakov pozdnyakov marked this pull request as ready for review October 30, 2019 10:26
Composing tiles from the GeoJSON features is an expensive
operation that might block UI thread on updating the `GeoJsonSource`
with the new data.

This change moves tile composing to the background thread and thus
unblocks the UI thread.
@pozdnyakov pozdnyakov force-pushed the mikhail_geojson_to_vt_in_background branch from 085e1d1 to 18e9331 Compare October 30, 2019 12:29
@pozdnyakov pozdnyakov removed the needs changelog Indicates PR needs a changelog entry prior to merging. label Oct 30, 2019
Copy link
Contributor

@LukasPaczos LukasPaczos left a comment

Choose a reason for hiding this comment

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

LGTM! 🚢

@pozdnyakov pozdnyakov merged commit 5b38cfe into master Oct 31, 2019
@pozdnyakov pozdnyakov deleted the mikhail_geojson_to_vt_in_background branch October 31, 2019 08:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android performance Speed, stability, CPU usage, memory usage, or power usage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants