Resource should not extend JsonSerializable

Azure / azure-cosmosdb-java

Java Async SDK for SQL API of Azure Cosmos DB

MIT License

54 stars 61 forks source link

Resource should not extend JsonSerializable #176

Closed christopheranderson closed 5 years ago

christopheranderson commented 5 years ago

Resource extending JsonSerializable gives a big back door into people being able to set id on ContainerSettings/other immutable properties. The only place it's useful for it to be extending anything is potentially Item(Settings).

christopheranderson commented 5 years ago

Scoped this out for v3. Closing for now.