Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Source Titok marketing: discovery method is too long (~ 40 seconds) #12107

Closed
Tracked by #11029
midavadim opened this issue Apr 18, 2022 · 0 comments · Fixed by #11378
Closed
Tracked by #11029

Source Titok marketing: discovery method is too long (~ 40 seconds) #12107

midavadim opened this issue Apr 18, 2022 · 0 comments · Fixed by #11378

Comments

@midavadim
Copy link
Contributor

midavadim commented Apr 18, 2022 •
edited
Loading

Problem:
discovery method is too long

Reason:
each initialization method in stream contains read method to get list of advertiser ids

Solution:
advertiser ids should be extracted only during read process.
@midavadim midavadim mentioned this issue Apr 18, 2022
Closed
9 tasks
@midavadim midavadim linked a pull request Apr 18, 2022 that will close this issue
🐛 Source Titok marketing: update schema. add unittest, update SAT, fix initialization for Audience streams #11378
Merged
40 tasks
@midavadim midavadim changed the title discovery method is too long (~ 40 seconds) Source Titok marketing: discovery method is too long (~ 40 seconds) Apr 18, 2022
@midavadim midavadim mentioned this issue Apr 19, 2022
Merged
40 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@midavadim