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

Jump to bottom

Suggestion: console for invalidating invalidate-report-data should take segment id as a parameter #15734

Closed
mikkeschiren opened this issue Mar 26, 2020 · 1 comment
Closed

Suggestion: console for invalidating invalidate-report-data should take segment id as a parameter #15734

mikkeschiren opened this issue Mar 26, 2020 · 1 comment
Labels
answered For when a question was asked and we referred to forum or answered it.

Comments

@mikkeschiren
Copy link
Contributor

I think it would help a lot if the console command for invalidating report data would take the segment id, as an option.

So instead of:
/console core:invalidate-report-data --segment='eventCategory=@StringFoo;visitIp!=127.0.0.1'

We could have:
/console core:invalidate-report-data --segment-id=2

Also makes sense when the core:archive command takes the segment-id as a parameter, a.k.a:
./console core:archive --force-idsegments=2

If this is a good idea, I could look into it and do a pull request. For non breaking changes we could keep the segment definition option still.
@tsteur
Copy link
Member

tsteur commented Mar 26, 2020

Thanks for creating the issue @mikkeschiren This is a duplicate of #14763 Be great to follow that issue

@tsteur tsteur closed this as completed Mar 26, 2020
@tsteur tsteur added the answered For when a question was asked and we referred to forum or answered it. label Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
answered For when a question was asked and we referred to forum or answered it.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tsteur @mikkeschiren