new Select to collect ConceptElements from values #2606
Conversation
awildturtok
force-pushed
the
feature/selects-on-concept-columns
branch
from
e86fbea to
b49f15c
Compare
| @JsonBackReference | ||
| @JsonIgnore | ||
| @Setter | ||
| private TreeConcept concept; |
There was a problem hiding this comment.
Du kannst hier auch getHolder() nutzen, dann ist es eine Backreference weniger. Da ist die Typsierung zwar nicht so gut, aber vielleicht lässt sich da TreeConcept implizit erzwingen
There was a problem hiding this comment.
Das wäre dann ja auch nur eine Validierung, gute Idee.
| @Override | ||
| public Aggregator<?> createAggregator() { | ||
| if(resolved){ | ||
| return new ConceptElementsAggregator(concept); | ||
| } | ||
|
|
||
| return new ConceptValuesAggregator(concept); | ||
| } |
There was a problem hiding this comment.
Ist es nicht besser zwei Select CPSTypes zu erstellen anstatt der Fallunterscheidung?
There was a problem hiding this comment.
Oder wäre es möglich anstatt des ConceptValuesAggregator einen InternToExternMapper zu nutzen und dann auf Manager-Seite das Mapping zu den Values zu machen?
There was a problem hiding this comment.
Dann müsste man da die Baumlogik implementieren
There was a problem hiding this comment.
Ist es nicht besser zwei Select CPSTypes zu erstellen anstatt der Fallunterscheidung?
Das weiß ich selber auch nicht. Ich finde hier wo sie echt fast das gleiche machen es als eins umzusetzen besser weil man dann in der conf nicht nachdenken muss welche es gibt, sondern nur die Frage stellen muss ob man die Baumelemente oder die Werte haben möchte.
...data/conquery/models/query/queryplan/aggregators/specific/value/ConceptValuesAggregator.java
Outdated
Show resolved
Hide resolved
awildturtok
changed the base branch from
feature/element-id-in-concept-column
to
develop
| @JsonIgnore | ||
| @ValidationMethod(message = "Holder must be TreeConcept.") | ||
| public boolean isHolderTreeConcept() { | ||
| return getHolder().findConcept() instanceof TreeConcept; | ||
| } |
| @Data | ||
| public class ConceptColumnSelect extends UniversalSelect { | ||
|
|
||
| private boolean resolved = false; |
There was a problem hiding this comment.
Würde ich umbenennen, z.B. in outputId. resolved heißt im code eigentlich, dass eine id aufgelöst wurde. hier heißt es das man eine Id bekommt. Das ist gut fürs hirntraining :D
There was a problem hiding this comment.
resolved heißt im code eigentlich, dass eine id aufgelöst wurde. hier heißt es das man eine Id bekommt.
Aber hier wird doch auch die resolved-te id ausgegeben, das passt doch zusammen oder?
Was hälst du von asIds ? Dann steht das da auch in der conf ordentlich drin
No description provided.