-
Notifications
You must be signed in to change notification settings - Fork 266
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ADD keysize related limitation in documentation #1899
Conversation
## Entity fields size limitation | ||
|
||
Due to underlying DB limitations (see details [here](https://github.com/telefonicaid/fiware-orion/issues/1289)), | ||
the combined size (sum) of entity id, entity type and entity service path cannot exceed 1014 characters. If you |
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 read the 1014 and I thought to myself ... that's a typo!
Then I remembered, 10 bytes of overhead somewhere, I looked it up myself the other day! :-)
However, practically everybody will assume it's a typo, unless a short explanation of those 'lost 10 bytes' is given.
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.
Fixed in commit d90a46f
@@ -56,7 +56,7 @@ The following table provides information about the last Orion version supporting | |||
|
|||
| **Removed feature** | **Last Orion version supporting feature** | **That version release date** | | |||
|----------------------------------------------------------------------------|-------------------------------------------|---------------------------------| | |||
| XML API | Not yet removed (planned: 0.28.0) | February 29th, 2016 | | |||
| XML API | 0.28.0 | February 29th, 2016 | |
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.
Not actually related with this PR, but a small modification that was pending...
LGTM |
ADD keysize related limitation in documentation
Documentation pieces related with issue #1289