Added support for accessing non partitioned collections using V3 SDK

srinathnarayanan commented 5 years ago

Exposed PartitionKey.None object which corresponds a PartitionKeyInternal object where components=null
Added check for systemkey in PartitionKeyDefinition
Bumped API version to 2018-12-31
Added checks for null PartitionKeyInternal components in the places where it is accessed (DocumentQueryExecutionContextFactory and AddressResolver)
Modified partition key extraction in case pk doesn't exists in the document. Earlier Undefined.Value was added, now the Value added depends on whether the pk is a system key or not.
Removed partitionKey header modification in getHeaders as the partitionKey specified now (like PartitionKey.None) need not necessarily be serializable.
Added test to create non partitioned collection with older API version using REST, and access it with SDK

Other changes:

moderakh commented 5 years ago

PartitionKeyInternal is an internal construct in com.microsoft.azure.cosmosdb.internal.routing package which is an internal package.

This PR is making it public. The existing public methods in PartitionKeyInternal are really meant to be used internally. for example

   // PartitionKeyInternal
    public List<IPartitionKeyComponent> getComponents() {
        return this.components;
    }

moderakh commented 5 years ago

This is exposing internal API as public API @srinathnarayanan can we talk when you get a chance?

christopheranderson commented 5 years ago

@srinathnarayanan & @kushagraThapar will clean up any lingering test issues in another PR, mostly need this in for the OM changes.

Azure / azure-cosmosdb-java