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

Review post-conditions for layer load failure - orphaned data #918

Closed
rob-metalinkage opened this issue Aug 5, 2017 · 1 comment
Closed
Labels
OBE Temp label to manage the back log of old/inapplicable issues.

Comments

@rob-metalinkage
Copy link
Contributor

On failing to successfully load data, layer references appear in both osgeo_importer_uploadlayer and (potentially a subset of layers) in layers_layer.

Review what should happen and implement test cases to make sure expected behaviour does happen reliably. should cover bad data and failure in long running tasks, possibly user cancellation of tasks - whatever system behaviours should leave data in a known state and not proliferate orphaned data.

Restarting load process may mean we want to keep things around - in which case there needs to be UI support for finding, fixing and cleaning, and system admin support for expiring failed layers.

@emilyashley
Copy link
Member

What's the status on these failed layers say in osegeo importer uploadlayer?

It's worth noting that "Uploading" a layer and "Importing" a layer are different actions called different things. Which one are you talking about? (or both?)

@lhcramer lhcramer added the OBE Temp label to manage the back log of old/inapplicable issues. label May 2, 2018
@Coop56 Coop56 closed this as completed May 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OBE Temp label to manage the back log of old/inapplicable issues.
Projects
None yet
Development

No branches or pull requests

4 participants