yarpc / yab

96%

master: 96%

LAST BUILD ON BRANCH fix_digit_replacement

branch: fix_digit_replacement

CHANGE BRANCH

x

Reset

• fix_digit_replacement
• T538194
• abhinav-patch-1
• add_logger
• atomic
• auto-release
• baggage
• bench_connections_balance
• bench_err
• bench_warmup_err
• binary_file
• blacklist-tcurl
• bump_deps
• changelog
• cross_builds
• cut0.7.0
• cut7.0.0
• dash_3
• debug_conns
• debug_travis
• default_json
• default_non_string
• default_thrift
• deploy_on_go18
• dev
• disable_envelope
• disable_thrift_envelope_yaml
• docs
• empty_peerlist_err
• enable_trace
• enum_name
• fieldid
• fix_help
• glide_testimports
• go1.8
• go17
• h
• headers
• http_encoding
• http_err
• ignore_case
• improve_docs
• indent
• ini
• ini_test
• inmem_parse
• json_non_map
• latest_deps
• leak
• list
• list_thrift_err
• logging
• low_rps
• man
• master
• mention_online
• move_limiter
• multiplexed
• no_jaeger
• no_limit
• no_strict
• no_unquote
• no_warmup
• no_warmup_benchmark
• parse_bool
• peer_list_config
• peer_list_opts
• peerprovider_ctx
• pl-aliases
• pphttp
• pptemp
• proc-alias
• readme
• refactor_serializer
• release-fix
• release_0.6.1
• rename_peers
• report_on_stop
• rk
• rk-rd-sk
• rpc_prefix
• set-constants
• str_args
• tchannel-v1.2.0
• tchannel_app_err
• tchannel_peer
• tcurl_aliases
• template_args
• template_file_parse
• test_release
• test_to_wire
• thrift_envelope
• thrift_envelope_headers
• timeout
• timeout_race
• transport_options
• travis
• typedef-string
• update_deps
• update_man_osx
• update_thriftrw
• update_version
• upgrade-thriftrw
• use_root_spec
• v0.10.0
• v0.10.1
• v0.4.0
• v0.5.0
• v0.5.1
• v0.5.2
• v0.5.4
• v0.6.0
• v0.6.1
• v0.6.2
• v0.7.0
• v0.8.0
• v0.9.0
• v0.9.0-auto
• v0.9.0-auto2
• version
• yab-template
• yab-template2
• yaml
• yarpc-headers

Build # 707

Build Type

push

travis-ci

Committed by prashantv

Commit Message

Improve digit handling for errors during benchmark

Currently, we replace each digit with an X, if the digit is part of a
multi-digit number. However, this leads to some weird experiences, e.g.,
IPs are partially replaced (10.1.10.0 becomes XX.1.XX.0)

Even if we replace all digits with X, we still get multiple error
messages for different IPs (XX.X.X.X vs XX.XX.X.X). Instead, we should
replace all consecutive digits with a single X.

Run Details

2501 of 2614 relevant lines covered (95.68%)

2.57 hits per line