Implement SecondaryId conversion #3317
Conversation
| * selects the right column for the given secondaryId from a table | ||
| */ | ||
| @CheckForNull | ||
| private Column findSecondaryIdColumn(Table table) { |
There was a problem hiding this comment.
Den Teil hier habe ich in die Table Klasse verschoben und public gemacht, damit ich bei der Conversion auch darauf zugreifen kann.
backend/src/main/java/com/bakdata/conquery/apiv1/query/concept/filter/CQTable.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Duplizierte ungenutzte Klasse.
backend/src/main/java/com/bakdata/conquery/sql/conversion/model/Qualifiable.java
Show resolved
Hide resolved
...ava/com/bakdata/conquery/sql/conversion/cqelement/intervalpacking/AnsiSqlIntervalPacker.java
Show resolved
Hide resolved
| import org.jooq.Field; | ||
| import org.jooq.impl.DSL; | ||
|
|
||
| public class SelectsIds implements Qualifiable<SelectsIds> { |
There was a problem hiding this comment.
der Name ist etwas merkwürdig, sit das ein Typo?
There was a problem hiding this comment.
Meine Herleitung war: das sind die IDs für die Selects Klasse, also hab ich SelectsIds draus gemacht.
Du meinst Typo im Sinne dass es SelectIds heißen sollte?
Ansonsten würde ich anbieten:
SqlIdsSqlIdColumns
There was a problem hiding this comment.
Glaube SqlIdColumns gefällt mir am besten.
There was a problem hiding this comment.
@awildturtok Mir auch, hab die Klasse umbenannt. Ist sonst noch was offen?
There was a problem hiding this comment.
die Tests hast du aus den legacy tests kopiert oder? Wenn ja dann passt. Sonst würde ich nochmal drüber gehen.
There was a problem hiding this comment.
genau, sind 1 zu 1 übernommen, außer dass ich DATE_RANGE validity dates in start und end DATE column umgewandelt hab, sodass es für beide Dialekte passt.
| joinSecondariesCondition = leftIds.getSecondaryId().get().eq(rightIds.getSecondaryId().get()); | ||
| } | ||
| else { | ||
| joinSecondariesCondition = DSL.noCondition(); |
There was a problem hiding this comment.
invert und early return?
No description provided.