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

feat: improve perf json #1539

Merged
merged 4 commits into from Mar 14, 2024
Merged

feat: improve perf json #1539

merged 4 commits into from Mar 14, 2024

Conversation

cfabianski
Copy link
Collaborator

@cfabianski cfabianski commented Mar 13, 2024

Description

Only add schema_classification that have been classified as valid.
Otherwise, we end up with garbage in the report that we have to deal with when processing the jsonlines report afterwards.

Checklist

  • I've added test coverage that shows my fix or feature works as expected.
  • I've updated or added documentation if required.
  • I've included usage information in the description if CLI behavior was updated or added.
  • PR title follows Conventional Commits format

@cfabianski cfabianski requested a review from didroe March 13, 2024 10:20
@cfabianski cfabianski changed the title Feat/improve perf json processing feat: improve perf json Mar 13, 2024
@cfabianski cfabianski force-pushed the feat/improve-perf-json-processing branch from bebabe9 to c79af82 Compare March 13, 2024 21:56
@cfabianski cfabianski marked this pull request as ready for review March 13, 2024 21:56
@cfabianski cfabianski force-pushed the feat/improve-perf-json-processing branch from c79af82 to 9cda95c Compare March 13, 2024 21:58
@cfabianski cfabianski merged commit fef4de8 into main Mar 14, 2024
10 checks passed
@cfabianski cfabianski deleted the feat/improve-perf-json-processing branch March 14, 2024 09:22
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.

None yet

1 participant