Skip to content
This repository has been archived by the owner on Jun 7, 2024. It is now read-only.

ARUHA-2372 Store event disk size stats from nakadi #1073

Merged
merged 5 commits into from
Jun 25, 2019
Merged

Conversation

antban
Copy link
Contributor

@antban antban commented Jun 21, 2019

ARUHA-2372 Store event disk size stats from nakadi

@v-stepanov
Copy link
Contributor

@antban code looks good.
But I think KafkaTopicRepository.getSizeStats() and DiskUsageStatsJob need tests.

final Map<String, Long> actualResult = job.loadDiskUsage();
assertEquals(expectedResult, actualResult);
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

ping

Copy link
Contributor

Choose a reason for hiding this comment

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

you can configure your IDE to add that automatically


import java.util.List;

public class KafkaZookeeper {
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks more like BubukuZookeeper :)

@v-stepanov
Copy link
Contributor

v-stepanov commented Jun 24, 2019

@antban
Everything looks good. Fix that super major problem with endline at the end of file and I will approve.

@v-stepanov
Copy link
Contributor

👍

1 similar comment
@antban
Copy link
Contributor Author

antban commented Jun 25, 2019

👍

@antban antban merged commit 753677f into master Jun 25, 2019
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.

2 participants