Skip to content

remove: p-meta and p-tags - #1067

Merged
nitisht merged 2 commits into
parseablehq:mainfrom
nikhilsinhaparseable:remove-p-meta-tags
Jan 6, 2025
Merged

remove: p-meta and p-tags#1067
nitisht merged 2 commits into
parseablehq:mainfrom
nikhilsinhaparseable:remove-p-meta-tags

Conversation

@nikhilsinhaparseable

Copy link
Copy Markdown
Member

remove p-meta, p-tags from ingestion flow
parseable will not add these fields to the event

remove from static schema creation
remove from query flow

remove p-meta, p-tags from ingestion flow
parseable will not add these fields to the event

remove from static schema creation
remove from query flow
@coveralls

coveralls commented Jan 6, 2025

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 12628165684

Details

  • 12 of 18 (66.67%) changed or added relevant lines in 4 files are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.5%) to 11.877%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/handlers/http/modal/utils/ingest_utils.rs 0 2 0.0%
src/handlers/http/ingest.rs 9 13 69.23%
Files with Coverage Reduction New Missed Lines %
src/event/format/mod.rs 1 68.13%
src/query/mod.rs 1 37.57%
Totals Coverage Status
Change from base Build 12628128688: -0.5%
Covered Lines: 2356
Relevant Lines: 19837

💛 - Coveralls

@nitisht
nitisht merged commit 6fc4a5b into parseablehq:main Jan 6, 2025
parmesant pushed a commit to parmesant/parseable that referenced this pull request Jan 13, 2025
remove p-meta, p-tags from ingestion flow
parseable will not add these fields to the event

remove from static schema creation
remove from query flow
@nikhilsinhaparseable
nikhilsinhaparseable deleted the remove-p-meta-tags branch July 12, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants