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

Fix issue with duplicate tiles being read for File and Cassandra backends #2200

Merged
merged 2 commits into from May 15, 2017

Conversation

Projects
None yet
2 participants
@lossyrob
Member

lossyrob commented May 15, 2017

This nasty bug was caused by some unfortunate variable naming and a bad merge; tiles were being read in for these to backends per-partition. Our unit tests had only 1 partition per RDD so it was never caught.

Fixes #2140

@lossyrob lossyrob added this to the 1.1 milestone May 15, 2017

@lossyrob lossyrob merged commit bd22f55 into locationtech:master May 15, 2017

1 of 2 checks passed

ip-validation The pull request did not pass Eclipse validation. The following users have invalid Signed-off-by footers: rdemanuele@gmail.com
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@lossyrob lossyrob removed the in progress label May 15, 2017

@fosskers

This comment has been minimized.

Contributor

fosskers commented May 16, 2017

Does it happen in only those two backends?

@lossyrob

This comment has been minimized.

Member

lossyrob commented May 17, 2017

Yup, just these two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment