ref configs #6
ref configs #6
Conversation
config.go
Outdated
| for i := 0; i < val.NumField(); i++ { | ||
| fieldName := val.Type().Field(i).Name | ||
|
|
||
| valueField := val.FieldByName(fieldName) | ||
| valueField := reflect.ValueOf(config).Elem().FieldByName(fieldName) |
There was a problem hiding this comment.
А чего за пределы цикла ValueOf не вынес? Я же об этом еще в прошлом ПРе писал...
There was a problem hiding this comment.
я подумал, что дело только в перезатирании, ща перенесу
There was a problem hiding this comment.
По поводу перезатирания надо потестить, я не уверен что reflect.Value автоматом переписывает оригинальный конфиг V-типа (а просто копирует содержимое во время ValueOf и дальше перезаписывает чисто свою копию). Обычно для такого пишут res.Interface().(V) после всех модификаций, чтобы вернуть финальное значение.
There was a problem hiding this comment.
А может и переписывает, там под капотом вот такая фигня происходит e := (*emptyInterface)(unsafe.Pointer(&i)) где i это конфиг пайплайнера.
There was a problem hiding this comment.
я создал две структуры с одинаковым полями и закинул их туда, он нужную структуру вернул
jice36
force-pushed
the
feature/sink_kafka_click
branch
from
d407b98 to
9503ce8
Compare
No description provided.