Skip to content

Commit

Permalink
Letting backup status location be variable
Browse files Browse the repository at this point in the history
  • Loading branch information
zmarois committed Dec 7, 2017
1 parent c55ba8e commit 0216283
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,7 @@ public class PriamConfiguration implements IConfiguration {
private static final String CONFIG_BACKUP_CHUNK_SIZE = PRIAM_PRE + ".backup.chunksizemb";
private static final String CONFIG_BACKUP_RETENTION = PRIAM_PRE + ".backup.retention";
private static final String CONFIG_BACKUP_RACS = PRIAM_PRE + ".backup.racs";
private static final String CONFIG_BACKUP_STATUS_FILE_LOCATION = PRIAM_PRE + ".backup.status.location";
private static final String CONFIG_MULTITHREADED_COMPACTION = PRIAM_PRE + ".multithreaded.compaction";
private static final String CONFIG_STREAMING_THROUGHPUT_MB = PRIAM_PRE + ".streaming.throughput.mb";
private static final String CONFIG_STREAMING_SOCKET_TIMEOUT_IN_MS = PRIAM_PRE + ".streaming.socket.timeout.ms";
Expand Down Expand Up @@ -1115,7 +1116,7 @@ public String getFlushInterval() {

@Override
public String getBackupStatusFileLoc() {
return "backup.status";
return config.get(CONFIG_BACKUP_STATUS_FILE_LOCATION, "backup.status");
}

@Override
Expand Down

3 comments on commit 0216283

@arunagrawal84
Copy link

Choose a reason for hiding this comment

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

@zmarois Can you provide this in upstream for 3.x and 3.11? I think its worth it that others can configure this as well.

@zmarois
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Absolutely. I'll push anything upstream that you are game for.

@zmarois
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@arunagrawal84 Netflix#639 and Netflix#640 Filed for this change

Please sign in to comment.