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

Performance and refactor of Spark module #73

Closed
timrobertson100 opened this issue Oct 5, 2023 · 5 comments
Closed

Performance and refactor of Spark module #73

timrobertson100 opened this issue Oct 5, 2023 · 5 comments
Assignees

Comments

@timrobertson100
Copy link
Member

Data growth has meant that the backfill jobs regularly take 9+ hours with 600 cores which is now a concern. Additionally, the code is convoluted and now difficult to maintain. Explorations in this proof of concept look promising and should be completed to provide a drop-in replacement of the module.

Some design goals:

  • A drop-in replacement module (target Spark 2, but plan for Spark 3)
  • Aim for a significant performance increase using the same or fewer resources
  • Aim to remove scala code unless necessary
  • Aim for simpler code to grok
@timrobertson100 timrobertson100 self-assigned this Oct 5, 2023
@timrobertson100
Copy link
Member Author

A first working skeleton for this is here
(1.2hrs for EPSG:3857)

@timrobertson100
Copy link
Member Author

If building on M1, then installing the very old protoc can be done with these instructions and this patch

timrobertson100 added a commit that referenced this issue Oct 24, 2023
@timrobertson100
Copy link
Member Author

Now deployed to DEV and UAT. Vector tile server full of:

WARN  [2023-10-24 14:41:14,811+0000] [qtp400103862-48969] org.gbif.maps.common.filter.VectorTileFilters: Unexpected non integer metadata entry null:121

Presumably year should be something other than null (0 perhaps?)

@timrobertson100
Copy link
Member Author

Presumably year should be something other than null (0 perhaps?)

Confirmed as 0, and fixed in branch

timrobertson100 added a commit that referenced this issue Oct 26, 2023
timrobertson100 added a commit that referenced this issue Oct 26, 2023
@timrobertson100
Copy link
Member Author

Now in production

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant