Skip to content

LOTC 1490 bot insights patches#185

Open
kcorbett-hdx wants to merge 6 commits into
mainfrom
LOTC-1490-bot-insights-patches
Open

LOTC 1490 bot insights patches#185
kcorbett-hdx wants to merge 6 commits into
mainfrom
LOTC-1490-bot-insights-patches

Conversation

@kcorbett-hdx

Copy link
Copy Markdown
Collaborator

This addresses

  • Issue 1 — Malformed timestamp in Akamai DS2 (CANNOT_PARSE_INPUT, Code 27)
  • Issue 2 — Missing UDFs in Akamai DS2 transform (UNKNOWN_FUNCTION, Code 46) (already completed)
  • Issue 3 — Entire record passed as timestamp value (Akamai DS2)
  • Issue 4 — Null primary timestamp in CloudFront Firehose transform

Jira: https://hydrolix.atlassian.net/browse/LOTC-1490

@kcorbett-hdx kcorbett-hdx temporarily deployed to bundle-validator-env April 23, 2026 15:30 — with GitHub Actions Inactive
kevinborkman-hub and others added 2 commits April 23, 2026 11:47
Normalize settings.sample_data from array to single dict across all six
transforms so the deploy path's sample extractor actually ingests rows
(previously only tencent's dict form passed through; the five array-typed
transforms silently skipped both insert and verification). Also promote
default's rich 23-field external sample_data.json content into the
embedded transform copy so test ingests land a realistic row rather than
just a {timestamp} stub. Fix a Go-layout typo in tencent's primary format
(15:05:05Z -> 15:04:05Z).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@kcorbett-hdx kcorbett-hdx temporarily deployed to bundle-validator-env April 23, 2026 16:30 — with GitHub Actions Inactive
@kcorbett-hdx kcorbett-hdx mentioned this pull request Apr 23, 2026
"EdgeResponseContentType": "video/mp4",
"EdgeResponseStatus": 200,
"EdgeServerIP": null,
<<<<<<< HEAD

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's some merge conflict stuff in here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants