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 unused columns. Add resource restrictions to XSEDE job ingestor. #88

Merged
merged 1 commit into from
Apr 6, 2017

Conversation

smgallo
Copy link
Contributor

@smgallo smgallo commented Apr 6, 2017

Remove unused columns. Add resource restrictions to XSEDE job ingestor.

Description

Remove unused columns from XDCDB jobs aggregator. Add resource restrictions to HPC jobs aggregator so we don't remove those records from the aggregation table (i.e., OSG records are excluded from ingestion and are handled by their own ingestor so don't delete them from the aggregate table when handling the other HPC jobs).

Motivation and Context

Keeping ETL configurations consistent.

Tests performed

Re-ran ingestion.

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.

@smgallo smgallo added the Category:ETL Extract Transform Load label Apr 6, 2017
@smgallo smgallo added this to the v6.6.0 milestone Apr 6, 2017
@smgallo smgallo requested a review from plessbd April 6, 2017 18:18
@smgallo smgallo merged commit 4c465bb into ubccr:xdmod6.6 Apr 6, 2017
@smgallo smgallo deleted the etl/xdcdb-ingestor branch April 6, 2017 18:38
ryanrath pushed a commit to ryanrath/xdmod that referenced this pull request Apr 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:ETL Extract Transform Load
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants