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

Remove data files for deleted export requests #1030

Merged
merged 1 commit into from
Sep 10, 2019

Conversation

jtpalmer
Copy link
Contributor

Description

Updates data warehouse export batch process to remove files corresponding to deleted requests.

Motivation and Context

Don't want files that should be deleted to stick around indefinitely.

Tests performed

Tested manually.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project as found in the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@jtpalmer jtpalmer added bug Bugfixes autodoc:ignore Auto documentation ignore this PR Category:Data Warehouse Export Data Warehouse Export labels Sep 10, 2019
@jtpalmer jtpalmer added this to the 8.5.0 milestone Sep 10, 2019
@jtpalmer jtpalmer merged commit 1ef1ffa into ubccr:xdmod8.5 Sep 10, 2019
smgallo added a commit that referenced this pull request Sep 19, 2019
* Add file locking to Data Warehouse Export script (#1029)

* Remove data files for deleted export requests (#1030)

* update roadmap url (#1031)

* Bug fixes: Add mask and make chart name unique

* Refresh clears the search field

* Add help window for Dashboard Components

* Update profile editor for renamed rest interface.

* Update dashboard chart display algorithm

* Add help images for core dashboard components (#1044)

* Add help images for core dashboard components

* Minor text changes (#1045)

* Fix metric explorer metric and dimension display bug.

* Add ability to hide realm from the metric catalog

* Fix Usage UI Regression (#1053)

* Fix Usage UI Regression

Co-Authored-By:Benjamin D. Plessinger <ben@plessinger.us>
@jtpalmer jtpalmer deleted the dw-export-remove-old-exports branch November 5, 2019 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autodoc:ignore Auto documentation ignore this PR bug Bugfixes Category:Data Warehouse Export Data Warehouse Export
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants