diff options
author | Martin Czygan <martin.czygan@gmail.com> | 2021-05-11 00:00:46 +0200 |
---|---|---|
committer | Martin Czygan <martin.czygan@gmail.com> | 2021-05-11 00:00:46 +0200 |
commit | 581a044fcc49144afe04e11d1d4bb662ad595f6b (patch) | |
tree | 37020c5d4372ea212f4d5ec8dc39a4d399bd08cc /skate | |
parent | 4fda7f72a87a863641138126c0b0e117a9b93d7b (diff) | |
download | refcat-581a044fcc49144afe04e11d1d4bb662ad595f6b.tar.gz refcat-581a044fcc49144afe04e11d1d4bb662ad595f6b.zip |
skip-on-empty: switch to one indexed column
Diffstat (limited to 'skate')
-rw-r--r-- | skate/cmd/skate-map/main.go | 6 | ||||
-rw-r--r-- | skate/map.go | 1 |
2 files changed, 4 insertions, 3 deletions
diff --git a/skate/cmd/skate-map/main.go b/skate/cmd/skate-map/main.go index 4b30927..3682e7b 100644 --- a/skate/cmd/skate-map/main.go +++ b/skate/cmd/skate-map/main.go @@ -54,7 +54,7 @@ var ( extraValue = flag.String("x", "", "extra value to pass to configurable mappers") bestEffort = flag.Bool("B", false, "best effort") logFile = flag.String("log", "", "log filename") - skipOnEmpty = flag.Int("skip-on-empty", -1, "omit docs with empty value in given column (zero indexed)") + skipOnEmpty = flag.Int("skip-on-empty", 0, "omit docs with empty value in given column (one indexed)") help = `skate-map available mappers @@ -87,8 +87,8 @@ func main() { if mapf, ok := availableMappers[*mapperName]; !ok { log.Fatalf("unknown mapper name: %v", *mapperName) } else { - if *skipOnEmpty >= 0 { - mapf = skate.WithSkipOnEmpty(mapf, *skipOnEmpty) + if *skipOnEmpty > 0 { + mapf = skate.WithSkipOnEmpty(mapf, *skipOnEmpty-1) } if *keyPrefix != "" { mapf = skate.WithPrefix(mapf, *keyPrefix) diff --git a/skate/map.go b/skate/map.go index cf2933e..4a79c1c 100644 --- a/skate/map.go +++ b/skate/map.go @@ -90,6 +90,7 @@ func WithBestEffort(f Mapper) Mapper { } // WithSkipOnEmpty ignores results where the value at a given field is empty. +// One indexed. func WithSkipOnEmpty(f Mapper, index int) Mapper { return func(p []byte) ([][]byte, error) { fields, err := f(p) |