This sets a proper RedisException on the Message, but has the core exception as InnerException as you'd expect.

This also means the Message will have the info we add to it, rather than *sometimes* having it.

Race here

Lines up with behaviors elsewhere - we already have semantics for failures here...following along.

This is generally yhe behavior we want I think.

This is to ensure assumptions are correct. I believe what's happening if we're re-fetching the tie breaker key on a background interval which is landing in the profiler on the multiplexer. This likely the same symptom is affecting several tests. For example, LowAllocationEnumerable.

If this is correct, we should likely find a way to exclude the tie breaker (and any heartbeat activities) from the profiler...or give the profiler an option, but this would require changing the interface (bad) or we add info to a profiled command to indicate. Or we decide this is proper behavior (profiling everything) and we move the tests off DB 0 or something so we can easily exclude the tiebreaker GET.

Before these were left in the scheduler on every success, which piles up needlessly. This cancels them upon success.

It's only needed for a few tests, what we were doing was drastic overkill, especially with background concurrent GC in play these days.

This is mainly around proper FireAndForget as well as .Wait() elimination. More to do but this is a big first pass.

improve configuration and tracking of async timeouts; only enable the time-marking if it is async-enabled (and not F+F)

e.g. are we dropping script evals, or just the last n items in the profile?

Long overdue collapsing of the two suites

# Conflicts:
#	NRediSearch.Test/RediSearchTestBase.cs
#	NRediSearch/AggregationResult.cs
#	NRediSearch/AssemblyInfo.cs
#	NRediSearch/Client.cs
#	NRediSearch/NRediSearch.csproj

* start work on nreisearch aggregates

* port chunks of JRediSearch

* port the rest of aggregates and the highlight/summarize API

* Port QueryTest

* Import all of ClientTest from JRediSearch

* Add aggregate tests; normlaize tests

* NRediSearch : all tests passing

* add source prefix to a bunch of files

# Conflicts:
#	NRediSearch.Test/ExampleUsage.cs
#	NRediSearch/NRediSearch.csproj

* start work on nreisearch aggregates

* port chunks of JRediSearch

* port the rest of aggregates and the highlight/summarize API

* Port QueryTest

* Import all of ClientTest from JRediSearch

* Add aggregate tests; normlaize tests

* NRediSearch : all tests passing

* add source prefix to a bunch of files

This is both load and noise (and causes output truncation). This makes life a lot better for free.

Useful when hanging, but noisy when not. Disable for now, but leave in config for when needed readily.

Unsure why this fails on Appveyor...adding some logging.

Also removes the 4096 value and set it to always-zero, since that's the effective behavior until removal.

See https://github.com/dotnet/sdk/pull/2330 for details.