Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Индексы в Таблице значений #1211

Closed
dmpas opened this issue Oct 28, 2022 · 15 comments
Closed

Индексы в Таблице значений #1211

dmpas opened this issue Oct 28, 2022 · 15 comments
Assignees
@dmpas
Milestone
v2.0-RC4

Comments

@dmpas
Copy link
Collaborator

dmpas commented Oct 28, 2022

Индексы в Таблицу Значений и Дерево Значений добавляются, но никак не влияют на производительность.
Необходимо добавить использование индексов при поисках.
@dmpas dmpas self-assigned this Oct 28, 2022
@dmpas dmpas mentioned this issue Oct 28, 2022
Closed
@EvilBeaver
Copy link
Owner

@dmpas ты же вроде открывал пулреквест на это, не?

@dmpas
Copy link
Collaborator Author

dmpas commented Mar 7, 2023

@dmpas ты же вроде открывал пулреквест на это, не?

Не...

@EvilBeaver
Copy link
Owner

Мяу-мяу?

@dmpas
Copy link
Collaborator Author

dmpas commented Mar 15, 2023

Мяу-мяу?

@EvilBeaver Всё. Приступил.

dmpas added a commit that referenced this issue Mar 19, 2023
@dmpas
#1211: Интерфейс индексов таблицы значений.
9932551
dmpas added a commit that referenced this issue Mar 19, 2023
@dmpas
#1211: Использование индексов в лоб.
64a8564
dmpas added a commit that referenced this issue Mar 19, 2023
@dmpas
#1211: Оптимизации по заветам профайлера.
0dd9269
@EvilBeaver
Copy link
Owner

Мяу?

@dmpas
Copy link
Collaborator Author

dmpas commented Oct 27, 2023

@EvilBeaver всё будет. Сначала до NUnit бы добраться...

@EvilBeaver
Copy link
Owner

@dmpas не, NUnit гиблое дело, лучше индексы сначала. Пользы людям больше будет. Вон, в ОСени, например, сканирование типов на таблицах значений может ускориться на сколько-то миллисекунд

@tormozit
Copy link

tormozit commented Nov 2, 2023

Что в вашем шифре означает "мяу"?

@EvilBeaver
Copy link
Owner

Что в вашем шифре означает "мяу"?

У тебя что, кота никогда не было?

@dmpas
Copy link
Collaborator Author

dmpas commented Nov 4, 2023

Вот уже даже что-то начинает работать.

dmpas added a commit to dmpas/OneScript that referenced this issue Nov 4, 2023
@dmpas
EvilBeaver#1211 Первый вариант с зелёными тестами.
68de5bd
dmpas added a commit that referenced this issue Nov 4, 2023
@dmpas
#1211 Тесты
5ee2cf6
dmpas added a commit that referenced this issue Nov 6, 2023
@dmpas
#1211 Исправления после слияния.
163d85b
@EvilBeaver
Copy link
Owner

@dmpas привет! Можем надеяться на пулреквест?

@dmpas
Copy link
Collaborator Author

dmpas commented Jan 20, 2024

@dmpas привет! Можем надеяться на пулреквест?

@EvilBeaver да, постараюсь сегодня закинуть. Вчера решил вдруг аннотации в двойку перенести и как-то погряз...

dmpas added a commit to dmpas/OneScript that referenced this issue Jan 20, 2024
@dmpas
EvilBeaver#1211 Уменьшение когнитивки Сгруппировать.
0b02dd1 
По заветам сонара проведен рефакторинг метода.
dmpas added a commit that referenced this issue Jan 20, 2024
@dmpas
#1211 Очистка индексов перед тяжелыми операциями.
0ff6fda
dmpas added a commit that referenced this issue Jan 20, 2024
@dmpas
#1211 Типизированное исключение.
6fcc654 
Ошибка ненайденной колонки выделена отдельным исключением.
@EvilBeaver EvilBeaver moved this from Todo to Done in OneScript and all around Jan 21, 2024
@EvilBeaver EvilBeaver added this to the v2.0-RC4 milestone Jan 21, 2024
@dmpas
Copy link
Collaborator Author

dmpas commented Jan 21, 2024

В дереве-то ещё нет ничего.

@dmpas dmpas reopened this Jan 21, 2024
@EvilBeaver
Copy link
Owner

Ну... А я уже и ветку снёс

@dmpas dmpas changed the title Индексы в Таблице значений и Дереве значений Индексы в Таблице значений Jan 22, 2024
@dmpas
Copy link
Collaborator Author

dmpas commented Jan 22, 2024

в дереве значений нет индексов. Закрываем без него.

@dmpas dmpas closed this as completed Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmpas dmpas

Labels
None yet
Projects
OneScript and all around
Status: Done
Milestone
v2.0-RC4
Development

No branches or pull requests
3 participants
@EvilBeaver @dmpas @tormozit