The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Яндекс открыл код распределённой СУБД YDB, поддерживающей SQL, opennews (?), 19-Апр-22, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


19. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +1 +/
Сообщение от Аноним (19), 19-Апр-22, 20:00 
Мда, октрыл напопад файл, а у них в коде имена просто улёт:

https://github.com/ydb-platform/ydb/blob/main/ydb/core/cms/w...

constexpr const char *WALLE_CMS_USER = "Wall-E";

namespace NKikimr, прямо кикимора какая-то ))

Интересно, на чём они там в Яндексе ... ну вы поняли ))

Ответить | Правка | Наверх | Cообщить модератору

26. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +4 +/
Сообщение от Аноним (26), 19-Апр-22, 20:20 
Ну кроме стеба, а кусок кода неплохой, очень даже
заметь что constexpr const char *WALLE_CMS_USER = "Wall-E"; это совсем не #define WALLE_CMS_USER "Wall-E", как могло бы быть :)
здесь же - namespace NKikimr { namespace NCms { - неймсейсы есть, тоже в плюс
ну если дальше пробежаться по коду в том же каталоге, видно влияние Фаулера, грамотные названия классов, методов и переменных.
Ответить | Правка | Наверх | Cообщить модератору

68. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +/
Сообщение от Аноним (68), 19-Апр-22, 23:32 
Ага, а тестов - кот наплакал. Лично я боюсь такое даже ставить на потестировать.
Ответить | Правка | Наверх | Cообщить модератору

132. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +1 +/
Сообщение от Аноним (26), 20-Апр-22, 19:22 
файлы с суффиксом *_ut.cpp в каталогах - юнит тесты
Ответить | Правка | Наверх | Cообщить модератору

102. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +/
Сообщение от Аноним (102), 20-Апр-22, 09:08 
-#define
+namespace NKikimr { namespace NCms {
+constexpr const char *


Код "не плохой", код графоманский. Метрики "хорошести" - такие метрики.

Ответить | Правка | К родителю #26 | Наверх | Cообщить модератору

133. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +/
Сообщение от Аноним (26), 20-Апр-22, 19:55 
это уже надо внимательно смотреть, из минусов можно отметить изобилие мелких файлов с классами на две строчки и кучу вложенных каталогов - черт ногу сломит, как в java принято написано
Ответить | Правка | Наверх | Cообщить модератору

63. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +/
Сообщение от Аноним (63), 19-Апр-22, 23:04 
Не заглядывайте в https://github.com/ydb-platform/ydb/tree/main/build Почти все ссылки на внутренние сервисы ведут
Ответить | Правка | К родителю #19 | Наверх | Cообщить модератору

65. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +5 +/
Сообщение от Аноним (63), 19-Апр-22, 23:07 
Kikimr, судя по описаниям коммитов, это их внутреннее название YDB
Ответить | Правка | К родителю #19 | Наверх | Cообщить модератору

90. "Яндекс открыл код распределённой СУБД YDB, поддерживающей SQ..."  +/
Сообщение от n00by (ok), 20-Апр-22, 07:29 
> Мда, октрыл напопад файл, а у них в коде имена просто улёт:
> https://github.com/ydb-platform/ydb/blob/main/ydb/core/cms/w...
> constexpr const char *WALLE_CMS_USER = "Wall-E";

Очевидно, проекту больше лет, чем constexpr и некоторым экспертам, и изначально это был макрос.

Ответить | Правка | К родителю #19 | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру