Trigger calculate column position from table #3500
Conversation
| @@ -43,6 +48,10 @@ public class Table extends Labeled<TableId> implements NamespacedIdentifiable<Ta | |||
| @JsonManagedReference | |||
| private Column primaryColumn; | |||
|
|
|||
| @JsonIgnore | |||
| @Getter(AccessLevel.NONE) | |||
| private Object2IntMap<Column> columnPositions = new Object2IntOpenHashMap<>(); | |||
There was a problem hiding this comment.
Ich hatte es dir im vorherigen PR schon angemerkt, du kannst hier keine Map benutzen, das wird die Performance komplett tanken. Das muss in der Column als Feld gecached sein.
There was a problem hiding this comment.
Ah danke nochmal für den hinweis, dann dreh ich es um und mache einen kommentar dran
There was a problem hiding this comment.
Ich habe es jetzt umgebaut:
Eine Table wird nach der Deserializierung automatisch initialisiert, dabei werden die positions in den columns gesetzt.
thoniTUB
force-pushed
the
feature/calculate-column-position-in-table
branch
from
2cbb21f to
631ee79
Compare
thoniTUB
changed the title
|
Ist rot |
…n-in-table' into feature/calculate-column-position-in-table # Conflicts: # backend/src/main/java/com/bakdata/conquery/io/storage/MetaStorage.java # backend/src/main/java/com/bakdata/conquery/mode/ManagerProvider.java # backend/src/main/java/com/bakdata/conquery/models/config/XodusStoreFactory.java # backend/src/test/java/com/bakdata/conquery/io/AbstractSerializationTest.java # backend/src/test/java/com/bakdata/conquery/io/jackson/serializer/IdRefrenceTest.java # backend/src/test/java/com/bakdata/conquery/util/extentions/MetaStorageExtension.java # backend/src/test/java/com/bakdata/conquery/util/extentions/NamespaceStorageExtension.java # backend/src/test/java/com/bakdata/conquery/util/extentions/WorkerStorageExtension.java
…-parsing' into feature/calculate-column-position-in-table
|
Es war unter anderem rot weil der ColumnValueAggregator die Werte in einer anderen Reihenfolge ausgegeben hat. |
There was a problem hiding this comment.
https://stackoverflow.com/a/57757055
Ich habe gerade noch den weg gesehen, der weniger aufwändig ist, sobald es implementiert ist.
…zation-upon-parsing Feature/integrate object initialization upon parsing
…ion-in-table # Conflicts: # backend/src/test/java/com/bakdata/conquery/io/AbstractSerializationTest.java # backend/src/test/java/com/bakdata/conquery/io/jackson/serializer/SerializationTestUtil.java # backend/src/test/java/com/bakdata/conquery/util/extensions/NamespaceStorageExtension.java
thoniTUB
merged commit 91f8f38
into
feature/integrate-object-initialization-upon-parsing
Inverse the inversion