appbaseio / appbaseio/abc

Enhancement: abc import should report indexing failures

Open
#172 0 comments 0 reactions 1 assignee Assigned to @harsh-2711 View on GitHub
difficulty/easy enhancement
Dominant language
Go
Stars
471
Forks
47
PR merge metrics
No merged PRs in 30d

Description

`abc import` logs currently don't show the number of records that failed to index. This behavior can be easily supported as the underlying bulk API reports the failures.

A sample output line for this can be:

- When there are failures: `10 out of 4654 total records failed to index`
- When indexing succeeds: `4654 total records indexed`.

Note: We only report total records when a data source import is divided across multiple bulk requests. We should correspondingly also report total failures per import command.

**Version Info**:
Version: 1.0.0-alpha.0
Variant: !oss
Go version: go1.11.10
OS: linux
Arch: amd64

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.