More upgrade docs, minor fixes in a couple of places.
Warn in neo4j-shell when using -readonly without -path
Corrected warning about java runtime
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
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
Merge pull request #1644 from chrisvest/shell-readonly-warnings

[AFTER GA] Warn in neo4j-shell when using -readonly without -path
Merge pull request #1699 from tinwelint/2.0-timeout-payload-fix

Passes correct payload for phase1Timeout messages
Merge pull request #1684 from lassewesth/pushstrategytestfix

using indexes instead of server ids makes test deterministic
Merge pull request #1686 from lassewesth/batchinsertertests

inserted bachmanm's tests for property reset in batchinserter
Merge pull request #1677 from lassewesth/warningsaboutjavaruntimeandversion

Corrected warning about java runtime
Make sure plan descriptions of entity producers / start items contain a producerType entry

o Also adds list of identifiers to these descriptions
o Renames StartItem.name to producerType
Document how Cypher compares values by equality and for ordering