mirror of
https://github.com/zitadel/zitadel.git
synced 2025-05-06 09:36:46 +00:00
fix(fields): add index to improve search by aggregate (#8267)
# Which Problems Are Solved During performance testing of the `eventstore.fields` table we found some long running queries which searched for the aggregate id. # How the Problems Are Solved A new index was added to the `eventstore.fields`-table called `f_aggregate_object_type_idx`. # Additional Changes None # Additional Context - Table was added in https://github.com/zitadel/zitadel/pull/8191 - Part of https://github.com/zitadel/zitadel/issues/7639
This commit is contained in:
parent
ab7c484157
commit
23bebc7e30
27
cmd/setup/31.go
Normal file
27
cmd/setup/31.go
Normal file
@ -0,0 +1,27 @@
|
|||||||
|
package setup
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
_ "embed"
|
||||||
|
|
||||||
|
"github.com/zitadel/zitadel/internal/database"
|
||||||
|
"github.com/zitadel/zitadel/internal/eventstore"
|
||||||
|
)
|
||||||
|
|
||||||
|
var (
|
||||||
|
//go:embed 31.sql
|
||||||
|
addAggregateIndexToFields string
|
||||||
|
)
|
||||||
|
|
||||||
|
type AddAggregateIndexToFields struct {
|
||||||
|
dbClient *database.DB
|
||||||
|
}
|
||||||
|
|
||||||
|
func (mig *AddAggregateIndexToFields) Execute(ctx context.Context, _ eventstore.Event) error {
|
||||||
|
_, err := mig.dbClient.ExecContext(ctx, addAggregateIndexToFields)
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
|
||||||
|
func (mig *AddAggregateIndexToFields) String() string {
|
||||||
|
return "31_add_aggregate_index_to_fields"
|
||||||
|
}
|
1
cmd/setup/31.sql
Normal file
1
cmd/setup/31.sql
Normal file
@ -0,0 +1 @@
|
|||||||
|
CREATE INDEX CONCURRENTLY IF NOT EXISTS f_aggregate_object_type_idx ON eventstore.fields (aggregate_type, aggregate_id, object_type);
|
@ -114,6 +114,7 @@ type Steps struct {
|
|||||||
s28AddFieldTable *AddFieldTable
|
s28AddFieldTable *AddFieldTable
|
||||||
s29FillFieldsForProjectGrant *FillFieldsForProjectGrant
|
s29FillFieldsForProjectGrant *FillFieldsForProjectGrant
|
||||||
s30FillFieldsForOrgDomainVerified *FillFieldsForOrgDomainVerified
|
s30FillFieldsForOrgDomainVerified *FillFieldsForOrgDomainVerified
|
||||||
|
s31AddAggregateIndexToFields *AddAggregateIndexToFields
|
||||||
}
|
}
|
||||||
|
|
||||||
func MustNewSteps(v *viper.Viper) *Steps {
|
func MustNewSteps(v *viper.Viper) *Steps {
|
||||||
|
@ -159,6 +159,7 @@ func Setup(ctx context.Context, config *Config, steps *Steps, masterKey string)
|
|||||||
steps.s28AddFieldTable = &AddFieldTable{dbClient: esPusherDBClient}
|
steps.s28AddFieldTable = &AddFieldTable{dbClient: esPusherDBClient}
|
||||||
steps.s29FillFieldsForProjectGrant = &FillFieldsForProjectGrant{eventstore: eventstoreClient}
|
steps.s29FillFieldsForProjectGrant = &FillFieldsForProjectGrant{eventstore: eventstoreClient}
|
||||||
steps.s30FillFieldsForOrgDomainVerified = &FillFieldsForOrgDomainVerified{eventstore: eventstoreClient}
|
steps.s30FillFieldsForOrgDomainVerified = &FillFieldsForOrgDomainVerified{eventstore: eventstoreClient}
|
||||||
|
steps.s31AddAggregateIndexToFields = &AddAggregateIndexToFields{dbClient: esPusherDBClient}
|
||||||
|
|
||||||
err = projection.Create(ctx, projectionDBClient, eventstoreClient, config.Projections, nil, nil, nil)
|
err = projection.Create(ctx, projectionDBClient, eventstoreClient, config.Projections, nil, nil, nil)
|
||||||
logging.OnError(err).Fatal("unable to start projections")
|
logging.OnError(err).Fatal("unable to start projections")
|
||||||
@ -182,6 +183,7 @@ func Setup(ctx context.Context, config *Config, steps *Steps, masterKey string)
|
|||||||
steps.s1ProjectionTable,
|
steps.s1ProjectionTable,
|
||||||
steps.s2AssetsTable,
|
steps.s2AssetsTable,
|
||||||
steps.s28AddFieldTable,
|
steps.s28AddFieldTable,
|
||||||
|
steps.s31AddAggregateIndexToFields,
|
||||||
steps.FirstInstance,
|
steps.FirstInstance,
|
||||||
steps.s5LastFailed,
|
steps.s5LastFailed,
|
||||||
steps.s6OwnerRemoveColumns,
|
steps.s6OwnerRemoveColumns,
|
||||||
|
Loading…
x
Reference in New Issue
Block a user