4paradigm / 4paradigm/OpenMLDB

make `enable_distsql` defaults on

Abierto
#3,544 0 comentarios 0 reacciones 2 asignados Reclamado por @lqy222 Ver en GitHub
bug high-priority test
Lenguaje dominante
C++
Estrellas
1.7k
Forks
331
Merge medio
12 d 12 h
PR fusionados (30 d)
1

Descripción

tablet server uses this flag `enable_distsql` to enable cluster optimization https://github.com/4paradigm/OpenMLDB/blob/b685e648e4467d86ce1ad8233b9428fb50313e09/src/tablet/tablet_impl.cc#L210-L214. It is on with release configurations: `release/conf/tablet.flags.template`, but no in `src/flags.cc`. Which as a results, won't enabled in mini_cluster tests, e.g `sql_sdk_test`.

There is a PR previously #3295, but turns out breaks existing tests. It means there exists cases that is OK with request mode, but not request(cluster) mode.

#3531 is a example, deployment tests not fail for those tests, simply because `enable_distsql` not turned on.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

The flag `enable_distsql` is defined in src/flags.cc and used in src/tablet/tablet_impl.cc lines 210-214. Check release/conf/tablet.flags.template for its default in release builds. Run mini_cluster tests like sql_sdk_test to see failures when the flag is enabled, as indicated by previous PR #3295. Investigate why certain tests pass in request mode but fail in request(cluster) mode, referencing issue #3531 as an example.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cpp
Área
backend, testing
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.