Feature/missing hana tests#3886
Conversation
thoniTUB
left a comment
thoniTUB
left a comment
There was a problem hiding this comment.
Hier werden viele Tests gelöscht, ist das korrekt?
| // Would prefer this to be `is not null`, but hana does not support that for fields | ||
| Field<String> isResolved = innerPrimaryColumn.as(IS_RESOLVED_ALIAS); |
There was a problem hiding this comment.
Warum kein boolean mehr und muss dann nicht auch das SQL Beispiel aus dem Kommentar angepasst werden?
There was a problem hiding this comment.
steht doch im Kommentar direkt drüber? Ich hab keine Syntax / konstrukt gefunden die für Hana funktioniert (Also einfach is not null meckert hana obwohl es eigentlich eine gültige Hana Expression ist). Ich schaue mal, ob ein case-when funktioniert.
| return CQTableContext.builder() | ||
| .ids(ids) | ||
| .validityDate(tablesValidityDate) | ||
| .validityDate(Optional.of(tablesValidityDate)) |
There was a problem hiding this comment.
Hier auch das Optional killen?
| else if (preFinalSelects.getValidityDate().isEmpty()) { | ||
| // TODO i think this is unreachable? | ||
| return preFinalSelects.withValidityDate(functionProvider.allRange()); | ||
| } |
There was a problem hiding this comment.
Tabellen und Queries ohne ValidityDate sind doch valide. Warum wird hier denn eins gesetzt?
There was a problem hiding this comment.
ich muss mal schauen, ob das noch reachable ist. Der Grund warum es evtl nicht erreicht sein kann ist, weil ich versucht habe bei allen cases von anfang an ein ValidityDate (dann eben ein leeres) mitzureichen. Das vereinheitlicht sehr viel code, statt dass man es an jeder stelle separat behandeln muss.
| } | ||
| // not all DB vendors throw SQLExceptions | ||
| catch (SQLException | RuntimeException e) { | ||
| log.error("", e); |
There was a problem hiding this comment.
| log.error("", e); | |
| log.error("Query execution failed", e); |
There was a problem hiding this comment.
Sind die hier gelöschen schon durch andere Test abgedeckt?
2 participants
No description provided.