Data aggregation processors added; Issue #3: Stdin support added; Issue: #5 JSON config support added#7
Open
b0s-works wants to merge 3 commits into
Open
Data aggregation processors added; Issue #3: Stdin support added; Issue: #5 JSON config support added#7b0s-works wants to merge 3 commits into
b0s-works wants to merge 3 commits into
Conversation
added 2 commits
June 6, 2018 11:52
…nd countUnique Structure separated by two part: stdin receiver and files reader blocks Work isn't done, but needs a review, comments and observations to guide me to the right developing path Program became runable, but work isn't done. Status Early alpha
oleg-raev
reviewed
Jun 8, 2018
| "github.com/sirupsen/logrus" | ||
| ) | ||
|
|
||
| /* |
Owner
There was a problem hiding this comment.
Remove comment, specified by CSV file
oleg-raev
reviewed
Jun 8, 2018
| data := strings.Split(row, ";") | ||
| if len(data) < 1 || len(data[0]) < 13 { | ||
| return | ||
| func main() { |
Main function on top now ValueSaver interface renamed to DataProcessor summarizer.go.Storage type renamed to Result Summarizer processors structures( avg.go, count-unique.go, sum.go ) renamed File reader and StdIn functions moved to input.go Flag package used to determine using of «-stdin» command line parameter to change source to stdin Itoa and FormatInt used instead of Sprintf to convert int/int64 type numbers to string ProcessRow function now part of Storage type, so this func became shorter and more flexible
b0s-works
changed the title
b0s-works
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Added dynamic type data aggregation summarization methods: avg, sum and countUnique
Structure separated by two part: stdin receiver and files reader blocks
Work isn't done, but needs a review, comments and observations to guide me to the right developing path
Best Regards, Ivan