unistack-org/micro
4
0
Fork 2
Code Issues 18 Pull Requests Actions Packages Projects Releases 324 Wiki Activity

initial hasql support #407

Merged
vtolstov merged 8 commits from hasql into v4 2025-09-20 22:57:40 +03:00
Conversation 9 Commits 8 Files Changed 10 +1104 -1
vtolstov commented 2025-09-18 14:35:03 +03:00
Owner
Copy Link

closes #403

Signed-off-by: Vasiliy Tolstov v.tolstov@unistack.org

closes #403 Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
vtolstov added 1 commit 2025-09-18 14:35:04 +03:00
initial hasql support
Some checks failed
test / test (pull_request) Failing after 19m47s
Details
lint / lint (pull_request) Failing after 19m59s
Details
coverage / build (pull_request) Failing after 20m4s
Details
2886a7fe8a 
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
vtolstov added 1 commit 2025-09-18 14:35:12 +03:00
Merge branch 'v4' into hasql
Some checks failed
test / test (pull_request) Failing after 17m58s
Details
coverage / build (pull_request) Failing after 18m40s
Details
lint / lint (pull_request) Failing after 1m41s
Details
ea84ac094f
vtolstov added 1 commit 2025-09-18 17:16:25 +03:00
initial hasql support
Some checks failed
lint / lint (pull_request) Successful in 4m23s
Details
test / test (pull_request) Failing after 17m14s
Details
coverage / build (pull_request) Failing after 17m29s
Details
6c6916a050 
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
vtolstov added 1 commit 2025-09-19 11:26:21 +03:00
add default node state criterion
Some checks failed
lint / lint (pull_request) Successful in 2m36s
Details
test / test (pull_request) Failing after 17m14s
Details
coverage / build (pull_request) Failing after 17m30s
Details
e053eeac74 
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
vtolstov added 1 commit 2025-09-19 11:31:16 +03:00
move
Some checks failed
lint / lint (pull_request) Successful in 3m27s
Details
test / test (pull_request) Successful in 4m49s
Details
coverage / build (pull_request) Failing after 17m28s
Details
022326ddc4 
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
pugnack approved these changes 2025-09-19 12:00:44 +03:00
pugnack left a comment
Member
Copy Link

Выглядит неплохо. Крутая штука от yandex 👍

Кстати, а были мысли в интерфейс micro.Service вносить сущность базы данных?

Выглядит неплохо. Крутая штука от yandex 👍 Кстати, а были мысли в интерфейс micro.Service вносить сущность базы данных?
pugnack reviewed 2025-09-19 12:21:53 +03:00
cluster/sql/cluster.go Outdated
@@ -0,0 +13,4 @@
"golang.yandex/hasql/v2"
)
var errNoAliveNodes = errors.New("no alive nodes")
pugnack commented 2025-09-19 12:21:53 +03:00
Member
Copy Link

Hint: думаю, что стоит сделать ее public, чтобы обрабатывать этот кейс

Hint: думаю, что стоит сделать ее public, чтобы обрабатывать этот кейс
vtolstov commented 2025-09-20 22:28:00 +03:00
Author
Owner
Copy Link

готово

готово
vtolstov marked this conversation as resolved
pugnack reviewed 2025-09-19 12:22:24 +03:00
cluster/sql/cluster.go Outdated
@@ -0,0 +15,4 @@
var errNoAliveNodes = errors.New("no alive nodes")
func newSQLRowError() *sql.Row {
pugnack commented 2025-09-19 12:22:24 +03:00
Member
Copy Link

Хм, что-то очень "грязное". А по другому, видимо, никак, верно понял?

Хм, что-то очень "грязное". А по другому, видимо, никак, верно понял?
vtolstov commented 2025-09-19 23:22:30 +03:00
Author
Owner
Copy Link

да, почему-то в го решили, что по другому нельзя. Переделываю на sql/driver

да, почему-то в го решили, что по другому нельзя. Переделываю на sql/driver
vtolstov commented 2025-09-20 22:28:22 +03:00
Author
Owner
Copy Link

оставил для совместимости и заимлементил драйвер

оставил для совместимости и заимлементил драйвер
vtolstov marked this conversation as resolved
pugnack reviewed 2025-09-19 12:23:14 +03:00
cluster/sql/cluster.go Outdated
@@ -0,0 +228,4 @@
}
// ClusterOptions contains cluster specific options
type ClusterOptions struct {
pugnack commented 2025-09-19 12:23:14 +03:00
Member
Copy Link

Предлагаю для читаемости вынести в отдельный файлик cluster_options.go и перегнать туда все опции и билдеры и сгруппировать их там

Предлагаю для читаемости вынести в отдельный файлик cluster_options.go и перегнать туда все опции и билдеры и сгруппировать их там
vtolstov commented 2025-09-20 22:32:26 +03:00
Author
Owner
Copy Link

готово

готово
vtolstov marked this conversation as resolved
pugnack reviewed 2025-09-19 12:25:20 +03:00
cluster/sql/cluster.go Outdated
@@ -0,0 +478,4 @@
return nil
}
func (c *Cluster) SetConnMaxLifetime(td time.Duration) {
pugnack commented 2025-09-19 12:25:20 +03:00
Member
Copy Link

тут получается из-за hasql.Alive — выставит коннект только по доступным нодам, а те что в моменте будет недоступны не получат настройку и за ними придется следить, если появятся?

тут получается из-за hasql.Alive — выставит коннект только по доступным нодам, а те что в моменте будет недоступны не получат настройку и за ними придется следить, если появятся?
vtolstov commented 2025-09-19 23:23:10 +03:00
Author
Owner
Copy Link

Да, после переделки на sql/driver может лучше будет

Да, после переделки на sql/driver может лучше будет
vtolstov commented 2025-09-20 22:33:13 +03:00
Author
Owner
Copy Link

убрал, в случае кластера уже подаются инстансы где нужно сконфигурить пулы, для кластера нужно выключить это дело.

убрал, в случае кластера уже подаются инстансы где нужно сконфигурить пулы, для кластера нужно выключить это дело.
vtolstov marked this conversation as resolved
pugnack reviewed 2025-09-19 12:25:53 +03:00
cluster/sql/cluster.go Outdated
@@ -0,0 +479,4 @@
}
func (c *Cluster) SetConnMaxLifetime(td time.Duration) {
c.hasql.NodesIter(hasql.NodeStateCriterion(hasql.Alive))(func(n *hasql.Node[Querier]) bool {
pugnack commented 2025-09-19 12:25:53 +03:00
Member
Copy Link

IDE подсвечивает, что можно убрать лишнее приведение типа hasql.NodeStateCriterion(hasql.Alive) и просто отдавать hasql.Alive.

Аналогично в строках: 489, 496, 503, 511

IDE подсвечивает, что можно убрать лишнее приведение типа hasql.NodeStateCriterion(hasql.Alive) и просто отдавать hasql.Alive. Аналогично в строках: 489, 496, 503, 511
vtolstov commented 2025-09-20 22:33:26 +03:00
Author
Owner
Copy Link

убрал методы

убрал методы
vtolstov marked this conversation as resolved
pugnack reviewed 2025-09-19 12:26:18 +03:00
cluster/sql/cluster_test.go Outdated
@@ -0,0 +10,4 @@
"golang.yandex/hasql/v2"
)
func TestNewCluster(t *testing.T) {
pugnack commented 2025-09-19 12:26:18 +03:00
Member
Copy Link

Читать тяжело, что тестирует тест.

Кмк, тут лучше разбить на атомарные тесты, а не general flow. Могу взять в работу (проксирую стажерам или сам).

Читать тяжело, что тестирует тест. Кмк, тут лучше разбить на атомарные тесты, а не general flow. Могу взять в работу (проксирую стажерам или сам).
vtolstov commented 2025-09-19 23:23:40 +03:00
Author
Owner
Copy Link

Ок

Ок
vtolstov marked this conversation as resolved
vtolstov added 1 commit 2025-09-20 22:27:45 +03:00
implement driver
Some checks failed
lint / lint (pull_request) Successful in 2m43s
Details
test / test (pull_request) Successful in 4m53s
Details
coverage / build (pull_request) Failing after 17m18s
Details
f92e18897a 
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
vtolstov commented 2025-09-20 22:34:27 +03:00
Author
Owner
Copy Link

Выглядит неплохо. Крутая штука от yandex 👍

Кстати, а были мысли в интерфейс micro.Service вносить сущность базы данных?

неа, была мысль сделать micro-store-sql который позволяет забиндить имя запроса к запросу и сделать реализацию методов Read/Write и тп.

> Выглядит неплохо. Крутая штука от yandex 👍 > > Кстати, а были мысли в интерфейс micro.Service вносить сущность базы данных? неа, была мысль сделать micro-store-sql который позволяет забиндить имя запроса к запросу и сделать реализацию методов Read/Write и тп.
vtolstov commented 2025-09-20 22:35:04 +03:00
Author
Owner
Copy Link

добавил имплементацию драйвера для sql/driver чтобы можно было используя кластер создать драйвер и его использовать как *sql.DB

добавил имплементацию драйвера для sql/driver чтобы можно было используя кластер создать драйвер и его использовать как *sql.DB
vtolstov added 2 commits 2025-09-20 22:46:51 +03:00
split files bc011a2e7f 
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
split files
Some checks failed
lint / lint (pull_request) Successful in 2m27s
Details
test / test (pull_request) Failing after 17m13s
Details
coverage / build (pull_request) Failing after 17m30s
Details
614b740c56 
Signed-off-by: Vasiliy Tolstov <v.tolstov@unistack.org>
vtolstov merged commit ab051405c5 into v4 2025-09-20 22:57:39 +03:00
vtolstov deleted branch hasql 2025-09-20 22:57:41 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
pugnack
Labels
Clear labels
Kind/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
atolstikhin devstigneev pugnack vtolstov
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: unistack-org/micro#407
No description provided.
Delete Branch "hasql"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?