| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
**Было:** Prometheus decoder ограничивает длину значений меток метрик максимум 256 символами. Попытка передать метрику с более длинным значением метки приводит к исключению `TPrometheusDecodeException`.
**Стало:** Максимальная длина значения метки расширена до 1024 символов. Системы, которые генерируют метрики с более длинными значениями (например, с ID источников данных, URL, или сложных идентификаторов), теперь могут успешно передавать такие метрики в Unified Agent без ошибок.
**Практический эффект:** Улучшение совместимости с внешними системами мониторинга и приложениями, которые требуют передачи длинных меток. Существующие системы с метками < 256 символов продолжают работать без изменений.
---
## С точки зрения разработчика
### Структурные изменения
Расширение требовало изменения способа хранения длины строки в пуле коротких строк:
1. **Prometheus decoder** (library/cpp/monlib):
- `MAX_LABEL_VALUE_LEN`: 256 → 1024
2. **Short string pool** (logbroker/unified_agent/common):
- `MaxLabelSize`: 255 → 1024
- **Ключевое изменение:** Размер строки теперь хранится как `ui16` (2 байта) вместо `unsigned char` (1 байт)
- Было: `*slot->Payload() = static_cast<unsigned char>(s.size())`
- Стало: `memcpy(slot->Payload(), &size, sizeof(ui16))`
- Это позволяет хранить строки до 65535 символов (но логически ограничено на уровне валидации)
### Изменённые файлы
| Файл | Основные изменения |
|------|------------------|
| `prometheus_decoder.cpp` | `MAX_LABEL_VALUE_LEN: 256 → 1024` |
| `prometheus_decoder_ut.cpp` | +2 юнит-теста для новых граничных значений |
| `short_string_pool.h` | `MaxLabelSize: 255 → 1024`, operator[] использует ui16 для чтения размера |
| `short_string_pool.cpp` | Allocation и copy логика адаптирована для ui16 |
### Тестирование
Добавлены два новых юнит-теста:
- **LabelValueAtNewLimitIsAccepted:** Метрика с меткой ровно 1023 символа успешно парсится
```
"m{l=\"" + string(1023, 'a') + "\"} 1\n"
```
- **LabelValueOverNewLimitStillThrows:** Метрика с меткой ≥1024 символов выбрасывает исключение
```
"m{l=\"" + string(1024, 'a') + "\"} 1\n"
// → TPrometheusDecodeException: "trying to parse too long label value, size >= 1024"
```
### Безопасность и совместимость
- ✅ Обратно совместимо: строки < 256 символов работают так же
- ✅ Валидация: лимит 1024 проверяется на уровне Ref() в пуле
- ✅ Граничные случаи: добавлены тесты для 1023 и 1024 символов
- ✅ Выделение памяти: правильно рассчитано с учётом новых 2 байт для размера
- ✅ Я не нашел ситуации когда закодированная строка (длина + строка) улетала куда-то из пула и потом пыталась раскодироваться. Так что считаю, что миграции никакой не надо и все обратно совместимо. НАДЕЮСЬ Я НЕ ОШИБАЮСЬ
commit_hash:b30852954951bd8e65d69777a59ffeb58da76b47
|
