More upgrade docs, minor fixes in a couple of places.
Warn in neo4j-shell when using -readonly without -path
using indexes instead of server ids makes test deterministic
added constants to identify servers, for readability
inserted bachmanm's tests for property reset in batchinserter
re-enabled TestBatchInsert.shouldRunConstraintPopulationJobAtShutdown
s/Manaul/Manual/
Re-adding test stores
Merge pull request #1690 from chrisvest/fix-manaul-typo

s/Manaul/Manual/
Chaining SymbolTable between update actions MergePatternBuilder
Refactoring of Predicate.rewrite

This method didn't follow the signature from AstNode - instead of returning an Expression,
rewrite returned a Predicate. This meant that it is impossible to rewrite predicates that
are used as expressions to something other than Predicate.

When Cypher is caching expressions, it uses this rewriting. This is done when using
complicated expressions with sub-expressions that are aggregate expressions mixed with
sub-expressions that are not aggregating. In these cases, the rewriting would fail and
manifested itself through a "Missing identifier" error.

 The solution is to move the expectation that rewrite will return a Predicate to the user
 of the method.
Added tests for failing query mixing predicates and aggregates
Updates to profile information
Add support for reporting entity producers of merge node actions from profile

This is not complete update action support in 2.0 profiler but should help tracing query problems with MERGE
Filtering relationships by literal property in pattern predicates

(based on systay's 3db3d279b1f1aeb896d68f7b05c6b9becea58005)
Flushes the index updater after all updates have been applied

instead of after each update. Massive performance difference for large
transactions.
Removes a test that isn't complete and has been ignored over a year
Add a dummy javadoc jar to satisfy mvn central requirements for neo4j-browser.
Wait longer on leave
Better startup sequence in HAMS, to avoid NPEs
When an instance leaves the ProposerState logic for how many accepts to wait for is wrong. Fixed this.
Merge pull request #1639 from nawroth/minor

More upgrade docs, minor fixes in a couple of places.
Revert "Added improved debugging for incorrect message handling."

This reverts commit 95c36934adc838666aebb2200d7ee6a647d01a49.

Since there's no longer a specific need for it.
Passes correct payload for phase1Timeout messages

There was an issue where a phase1Timeout occured, but no payload passed
with it. The consumer of such a timeout used the payload and made it
available for later usage even, which then would unexpectedly fail.

Now, wherever applicable, the correct payload is sent with such timeout
messages.

co-author: Rickard Öberg, @rickardoberg