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.

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

Source Intercom: remove setting state of... logs from intercom sync #10250

Closed
Tracked by #10936
TylerButh opened this issue Feb 10, 2022 · 2 comments
Closed
Tracked by #10936

Source Intercom: remove setting state of... logs from intercom sync #10250

TylerButh opened this issue Feb 10, 2022 · 2 comments
Assignees
Labels
community type/enhancement New feature or request

Comments

@TylerButh
Copy link

Tell us about the problem you're trying to solve

When syncing Intercom data after each Records read I am seeing perhaps thousands of Setting state of ... stream to log messages. I'm not sure if those are logs from a separate task running simultaneously or if they are logs of the current task, but they flood my log file so that I can't download the logs because the file is too large or it makes debugging errors extremely difficult as I'm going through hundreds of thousands of lines of logs, the majority of which are these messages.

image

Describe the solution you’d like

It would be nice to either remove this log entry all together, it's not personally useful to me. Or perhaps have a single log entry for "Setting the state of 321 rows of company_segments stream to..." or maybe group the log entries by the updated_at time. If you notice from the screenshot they are all duplicate entries of the same time. Just grouping them by the time would eliminate thousands of entries.

Are you willing to submit a PR?

Not at the moment.

@misteryeo
Copy link
Contributor

Assignee should take a look at this and validate whether there's value in retaining individual log entries for setting state of... and determine if there's a cleaner UX we can implement here.

@lazebnyi lazebnyi self-assigned this Mar 23, 2022
@lazebnyi lazebnyi moved this to Ready for implementation (prioritized) in GL Roadmap Mar 23, 2022
@lazebnyi lazebnyi changed the title Remove setting state of... logs from intercom sync Source Intercom: remove setting state of... logs from intercom sync Mar 23, 2022
@lazebnyi lazebnyi moved this from Ready for implementation (prioritized) to Implementation in progress in GL Roadmap Mar 25, 2022
@lazebnyi
Copy link
Collaborator

Fixed in #9746

@lazebnyi lazebnyi moved this from Implementation in progress to Done in GL Roadmap Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community type/enhancement New feature or request
Projects
No open projects
Archived in project
Development

No branches or pull requests

5 participants