-
Notifications
You must be signed in to change notification settings - Fork 43
[5pt] Indexes: Community review #1943
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
Comments
По просьбе @NickVolynkin выкладываю сюда замечания (позже еще выложу по оставшейся теории из раздела) По индексам: https://www.tarantool.io/ru/doc/latest/book/box/indexes/
Я бы постаралась донести простыми словами, i.e.: В Tarantool индекс может быть также многокомпонентным, то есть состоять из двух или более полей кортежа. Для индекса типа TREE максимальное количество полей ключа - 255. И следующий абзац тоже бы упростить в повествовании, да и лучше бы попробовать в этом разделе донести информацию до пользователей как понял бы и простой дурак. :) Раздел "Создание индекса" предполагает собой пример, который и приведен, но тут есть фраза, предполагающая full table scan, и избыточна, на мой взгляд:
Вижу ссылку на типы индексируемых полей, но вот уже на ваше усмотрение: смотрите, с одной стороны было бы проще видеть эту информацию тут, хотя это скорее всего моя прерогатива. Типы индексов переместить бы чуть выше созданий и операций с ними. Правда, в операциях приведены secondary, partial ключи, но я не смогла найти что же такое partial в контексте Tarantool. |
R-TREE: https://www.tarantool.io/ru/doc/latest/book/box/indexes/ С одной стороны, для меня материал не кажется трудным, с другой, в первом абзаце раздела переиграла бы изложение следующим образом:
RTREE-индекс не может быть первичным и не может быть уникальным. - тавтология, на мой взгляд. Also, стоило бы ввести термины для distance и dimension, а далее по тексту использовать термины (или русский, или английский). Про итераторы в Примере 1 переделать бы таблицей со структурой по столбцам: итератор - как ищет - пример на основании итератора. BITSET: https://www.tarantool.io/ru/doc/latest/book/box/indexes/#bitset-indexes |
Index OperationsКажется недостаточно структурированным. Сейчас раздел состоит из следующих частей: По сути это описание возможностей tree индекса. Я бы предложил перенести это в раздел 'TREE indexes'. краткое описание и деталиЕщё есть отдельная страница space_object:create_index(), Предлагаю на главной минимизировать количество информации, а все детали перенести в create_index и описания итераторыНе совсем понятно является ли итератор независимой сущностью или итераторы работают по разному в каждом индексе ? |
Indexes: Community review
Product: Tarantool
Since: latest
Audience/target: all Tarantool users
Root document: https://www.tarantool.io/en/doc/latest/book/box/indexes/
https://www.tarantool.io/en/doc/latest/reference/reference_lua/box_space/create_index/
SME:
Peer reviewer: @NickVolynkin
Details
This is the forth step in revising our documentation about indexes. (See #1691, #1820 and #1871.)
Here we want to understand whether the revised documents became more informative and clear or not and what is left to write.
We have some concerns about the structure and the completeness of the provided information.
We suggest answering the following questions:
Please, leave your feedback here in comments.
Definition of done
To be done with: #1782 #1737
The text was updated successfully, but these errors were encountered: