Logs storage. Update common & parser. #12
Conversation
| } | ||
|
|
||
| "Test Log from HashMap" >> { | ||
| val (obj, hm) = genLog() |
There was a problem hiding this comment.
|
а какая цель рефакторинка? |
|
Главная проблема с которой столкнулся - равенство объектов помещённых и возвращенных из RethinkDB. Для macro добавил Вторая проблема: корректное формирование ReqlFunction1 для filter. А именно, запрет запросов типа "timestamp contains 'abc'". |
| level: Int, | ||
| env: String, | ||
| name: String, | ||
| timestamp: OffsetDateTime, |
There was a problem hiding this comment.
мб timestamp надо переименовать уже
|
досмотрю завтра. |
|
Изменены
и их комбинациями с другими query: |
|
клёво, это получается сделал частицу вот этого таска #19 |
| @@ -0,0 +1,11 @@ | |||
| package com.github.scalalab3.logs.common.domain | |||
|
|
|||
| case class Period(private val amountVal: Long = 0L, timeUnit: TimeUnit = Sec) { | |||
There was a problem hiding this comment.
почему private?
и зачем val, для case class аргументы будут всегда val
There was a problem hiding this comment.
Попытка сделать защиту от amountVal < 0, метод amount() тогда вернет 0.
|
zbs сделал |
|
Спасибо за code review :) |
|
пожалуйсто |
|
Обновлено. |
|
|
||
| val stringKeys: Seq[String] = typeOf[Log].members.collect { | ||
| case m: MethodSymbol | ||
| if m.isCaseAccessor & m.typeSignature.finalResultType =:= typeOf[String] => m.name.toString |
|
исправь там & -> && и я солью |
|
Исправлено. |
|
спасибо |
Simple logs db impl.
Some changes in parser and macro.