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

Remove redundant fields from /log payload #36

Open
tolbertam opened this issue Dec 13, 2017 · 1 comment
Open

Remove redundant fields from /log payload #36

tolbertam opened this issue Dec 13, 2017 · 1 comment
Assignees
@tolbertam
Milestone
0.13.0

Comments

@tolbertam
Copy link
Contributor

With #6, the entire native protocol frame is included in /log output. As part of that ticket, I left some existing now redundant fields in /log to not break existing users. We should remove these fields in the next release as they are now all present in the frame field:

  • type
  • query
  • consistency_level
  • serial_consistency_level
  • client_timestamp
@tolbertam tolbertam added this to the 0.9.0 milestone Dec 13, 2017
@tolbertam tolbertam self-assigned this Dec 13, 2017
@tolbertam
Copy link
Contributor Author

As part of this effort we will also need to refactor ActivityLogIntegrationTest as it depends on these fields currently.

@tolbertam tolbertam mentioned this issue Jan 5, 2018
Closed
@adutra adutra modified the milestones: 0.9.0, 0.12.0 Mar 22, 2021
@weideng1 weideng1 modified the milestones: 0.12.0, 0.13.0 Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tolbertam tolbertam

Labels
None yet
Projects
None yet
Milestone
0.13.0
Development

No branches or pull requests
3 participants
@adutra @weideng1 @tolbertam