-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft: multiple mapping values #3484
base: develop
Are you sure you want to change the base?
Conversation
…mapping-values # Conflicts: # backend/src/main/java/com/bakdata/conquery/models/types/ResultType.java
…multiple values are allow, we overwrite the entire string
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lass uns bitte nochmal einen anderen Ansatz finden.
Dadurch dass ResultTypes nicht mehr an Klassen gebunden sind wird es komplizierter die Result Renderer zu schrieben und auch zu verstehen.
Ich würde den CPSType-wegmachen PR gerne auch wieder reverten.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kannst du hier bitte noch mit TAB einrücken?
@@ -16,10 +16,11 @@ public interface InternToExternMapper extends NamespacedIdentifiable<InternToExt | |||
|
|||
boolean initialized(); | |||
|
|||
String external(String internalValue); | |||
ResultType<?> createMappedType(boolean list); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doku bitte für den flag
@Data | ||
public static class MappedMultiStringT extends ResultType<List<String>> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich denke mit diesen Klassen wird das ResultType System weniger definiert und wird sollten überlegen, wie man dass besser refactored, so dass die Logik für das erstellen des Ergebisses aus dem ResultType heraus wandert und näher am Select und seinem Mapping drannen ist.
Mir fällt das erst jetzt auf, dass wir durch abschaffen des CPSTypes in der Klasse, auch implizite Konventionen abgeschafft haben.
public String typeInfo() {
return "LIST[STRING]";
}
Ich glaube, dass das einen ziemlich krassen Rattenschwanz haben wird und wir das Feature nicht umsetzen sollten.