-
Notifications
You must be signed in to change notification settings - Fork 68
Conversation
|
||
} | ||
/* | ||
* Copyright (c) 2015, 2018 IBM Corp. All rights reserved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(C) or ©
* <p> | ||
* <b>Important:</b>Top-level field names that begin with the underscore character (_) are | ||
* <a href="https://console.bluemix.net/docs/services/Cloudant/api/document.html#documents" target="_blank"> | ||
* reserved</a> in Cloudant and must not be used. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should append something like...
If you wish to control the reserved properties e.g. _id
_rev
and _deleted
or map them to specific fields in your POJO then do not subclass com.cloudant.client.api.model.Document
, but use your own class instead.
CHANGES.md
Outdated
- [FIXED] An issue where `getReason()` returned an incorrect value for `Response` objects returned | ||
by `Database.bulk()`. | ||
- [FIXED] Issues retrieving deleted documents using an `AllDocsRequest`. | ||
- [BREAKING CHANGE] Added `_deleted` field to `Document`. This will break (de)serialisation for | ||
classes which sub-class `Document` and themselves declare a a field which implicitly or explicitly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicated a
in declare a a field
CHANGES.md
Outdated
- [FIXED] An issue where `getReason()` returned an incorrect value for `Response` objects returned | ||
by `Database.bulk()`. | ||
- [FIXED] Issues retrieving deleted documents using an `AllDocsRequest`. | ||
- [BREAKING CHANGE] Added `_deleted` field to `Document`. This will break (de)serialisation for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we can split this into two, since really there is an actual fix here and the "breaking change" is really more of a side effect for edge cases:
- [FIXED] Added missing `_deleted` field to `Document` model class.
- [BREAKING CHANGE] The fix for adding the `_deleted` field will break...
No description provided.