Skip to content
This repository has been archived by the owner on Apr 27, 2021. It is now read-only.

De-duplication flow #101

Merged
merged 4 commits into from Nov 7, 2017
Merged

De-duplication flow #101

merged 4 commits into from Nov 7, 2017

Conversation

olessia
Copy link

@olessia olessia commented Nov 1, 2017

  • Add deduplicator class.
  • Create a non-scheduled deduplication task: it will need to be modified manually in Azkaban to take a file.

Create a non-scheduled deduplication task: it will need to be modified manually in Azkaban to take a file.
@olessia
Copy link
Author

olessia commented Nov 1, 2017

wip

Copy link

@dterror-zz dterror-zz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good stuff

argsParser.parse(args, Params()) match {
case Some(params) =>
val backupPath = params.backupPath.replace("/$", "") // remove trailing slash
val targetPaths = params.targetPaths

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation


object TextWriter {
def apply(): TextWriter = new TextWriter()
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this necessary?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, not, strictly speaking, necessary.

@olessia olessia merged commit 83b10b9 into staging Nov 7, 2017
olessia pushed a commit that referenced this pull request Nov 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants