Skip to content

Commit

Permalink
Merge pull request #21 from aliyun/features/no_panic_seq
Browse files Browse the repository at this point in the history
perf: not panic for sequence featureview using featuredb
  • Loading branch information
bruceding authored Jul 2, 2024
2 parents 3dee591 + d0e9157 commit 83db6ae
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions domain/sequence_feature_view.go
Original file line number Diff line number Diff line change
Expand Up @@ -63,10 +63,6 @@ func NewSequenceFeatureView(view *api.FeatureView, p *Project, entity *FeatureEn
panic("deduplication_method invalid")
}

if p.OnlineDatasourceType == constants.Datasource_Type_FeatureDB || view.WriteToFeatureDB {
panic("sequence feature view not support featuredb yet")
}

daoConfig := dao.DaoConfig{
DatasourceType: p.OnlineDatasourceType,
PrimaryKeyField: sequenceFeatureView.userIdField,
Expand Down

0 comments on commit 83db6ae

Please sign in to comment.