Change ETL logging identification strings #1316
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Replaces the
DWI
identification strings with more specific names.Motivation and Context
The ident
DWI
has a specific meaning in the XSEDE ETL processes. It is used to identify thedw_extract_transform_load
process. There is a script that checks to make sure this process has been run in the past couple days. If all these other scripts use the same ident then the check script is unable to notify us that the ETL script has not been running.Tests performed
I haven't tested this. I'm not sure if there could be any side effects so I've requested reviews from many team members.
Types of changes
Checklist: