Skip to content

Hackathon demo: customer PII SQL review - #2

Merged
prantikmedhi merged 5 commits into
mainfrom
hackathon-real-demo
Apr 24, 2026
Merged

Hackathon demo: customer PII SQL review#2
prantikmedhi merged 5 commits into
mainfrom
hackathon-real-demo

Conversation

@prantikmedhi

Copy link
Copy Markdown
Owner

Live hackathon demonstration PR. Adds a SQL query against the OpenMetadata sandbox customer table so MetaReview can analyze a real pull request and post an automated review comment.

@github-actions

github-actions Bot commented Apr 24, 2026

Copy link
Copy Markdown

MetaReview Bot

Impact Score: MEDIUM (6.0/10)

Looks good, mostly documentation and example SQL changes.

Summary

This Pull Request updates project documentation, including README.md and SETUP.md, providing enhanced guidance and architectural details. It also introduces examples/customer_contact_review.sql, which contains a sample SQL query selecting data from acme_nexus_raw_data.acme_raw.crm.customers.

Risks

  • PII Exposure (Medium Risk - Score 6.0): The examples/customer_contact_review.sql query explicitly selects email and phone_number columns from the customers table. These fields are identified as Personally Identifiable Information (PII).
  • No usage of deprecated columns or tables was detected.

Impact

  • The changes primarily improve documentation clarity and provide a functional example to demonstrate MetaReview's features.
  • As the SQL is part of an example file, there is no direct operational impact on production data assets or downstream dependencies.

Recommended Action

Approve, with awareness that the example SQL explicitly handles PII. This is acceptable given its role as a demonstration, but users should be mindful when adapting such patterns for production environments.

@prantikmedhi
prantikmedhi merged commit 4e0b31c into main Apr 24, 2026
1 check passed
@prantikmedhi
prantikmedhi deleted the hackathon-real-demo branch April 26, 2026 19:28
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.

1 participant