Fix full outer joining when converting forms #3510
Conversation
There was a problem hiding this comment.
@thoniTUB kannst du dir bitte die Tests anschauen, ich verstehe von den Formularen nicht genug um das zu verifizieren.
| @@ -107,6 +107,19 @@ public ColumnDateRange as(String alias) { | |||
| ); | |||
| } | |||
|
|
|||
| public ColumnDateRange coalesce(ColumnDateRange right) { | |||
There was a problem hiding this comment.
das spricht dann eigentlich für mich, dass man das als konvention festlegen muss, und in der config enforcen muss. Ich kann Anwendern nicht erklären "die Konzepte könnt ihr nicht mixen, weil das schema es nicht hergibt" :D
There was a problem hiding this comment.
Und, könnte man nicht im Prinzip die EventFilter CTE zum normalisieren nehmen? (Wobei es dann vermutlich auf dual Column fallen würde)
There was a problem hiding this comment.
Und uns halt performance zerreissen würde
There was a problem hiding this comment.
Bzgl. erstem Kommentar: du meinst dass man single-column und dual-column nicht mixen kann oder? Das ist nicht ganz trivial, da ich durchaus innerhalb eines Dialektes beide Repräsentationen brauche, aber für die jeweiligen Methoden muss es dann ggf. die gleiche Art sein. Wenn ich aber z.B. eine DurationSum bei Postgres berechne, dann mache ich aus der single-column daterange ein start und end date damit ich die Differenz berechnen kann.
There was a problem hiding this comment.
Aber grundsätzlich würde ich mich auch interessieren, ob man auf den Postgres-Sonderweg verzichten kann und für alle die 2-column Lösung annimmt. Das würde den Code stellenweise extrem vereinfachen, aber über Performance kann ich natürlich aktuell noch keine Aussage treffen. Ich könnte das mal testweise ausprobieren wie sich z.B. Datumsaggregation dahingehend verhält.
There was a problem hiding this comment.
Ja, denselben Test gibt es an anderer Stelle und der war da scheinbar aus Versehen dupliziert 😄
| "/shared/vers_stamm.table.json", | ||
| "/tests/form/shared/abc.table.json" |
There was a problem hiding this comment.
Mich wundert etwas, dass die Pfade hier unterschiedlich anfangen und es trotzdem funktioniert
There was a problem hiding this comment.
Das ist unschön gelöst, ich musste teilweise die shared Fixtures anpassen. Es gibt zwei shared-Ordner, einen für SQL- und einen für Worker-Mode.
Sollte "Absolutes Export Formular - alle Konzepte werden verUNDet evaluiert" beheben.