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

Added delta-filters #998

Merged
merged 2 commits into from
Jun 20, 2024
Merged

Added delta-filters #998

merged 2 commits into from
Jun 20, 2024

Conversation

olsen232
Copy link
Collaborator

@olsen232 olsen232 commented Jun 19, 2024

Backwards compatible in that they do nothing unless you specify --delta-filter=
A bit like minimal-patches which we implemented a couple years back, but more powerful - more general-purpose - to avoid having to implement them again at some point.
Basically, lets a consumer of JSON or JSON-LINES diff opt in or out of any of the following:

  • inserts
  • updates (old values)
  • updates (new values)
  • deletes
    But still see where the filtered out data is.

Useful for generating changeset views of the data that don't need to show old values.

Checklist:

  • Have you reviewed your own change?
  • Have you included test(s)?
  • Have you updated the changelog?

Backwards compatible in that they do nothing unless you specify
--delta-filter=<some-value>
A bit like minimal-patches which we implemented a couple years back,
but more powerful - more general-purpose - to avoid having to implement
them again at some point.
Basically, lets a consumer of JSON or JSON-LINES diff opt in or out
of any of the following:
- inserts
- updates (old values)
- updates (new values)
- deletes
But still see where the filtered out data is.

Useful for generating changeset views of the data that don't need to show old values.
@olsen232 olsen232 requested a review from craigds June 19, 2024 05:08
Note that you can't currently apply a delta-filter that is missing
old revisions, this can be re-added later if needed.
The replacement supports slightly different things to the original
(if it was exactly the same there would be no point replacing it)
@olsen232 olsen232 merged commit cb74bd6 into master Jun 20, 2024
35 checks passed
@olsen232 olsen232 deleted the delta-filters branch June 20, 2024 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants