[TheL0L]

Related Issues

related: XSUP-78274

Description

Bound the connectivity test query to the last hour.

[content-bot]

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

• @marketplace-ai-reviewer start review - Initiate a full AI code review
• @marketplace-ai-reviewer re-review - Incremental review for new commits

[marketplace-ai-reviewer]

🤖 Analysis started. Please wait for results...

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
Packs/CortexDataLake/Integrations/CortexDataLake
CortexDataLake.py	754	252	66%	138–139, 249–253, 256–257, 260, 264, 272, 274, 280, 283, 307–311, 314–316, 322–323, 386–387, 442–447, 449–451, 460–464, 466–467, 491, 493–495, 497, 530, 532–534, 537–540, 542–545, 552, 559–563, 565–570, 572, 584, 588, 591, 604–606, 752, 806, 866, 921, 1112–1117, 1152–1153, 1217, 1220, 1240, 1243, 1245–1247, 1286–1287, 1289, 1300, 1302, 1304, 1306–1308, 1310–1311, 1355–1359, 1445–1448, 1453–1455, 1539, 1556–1559, 1561–1562, 1564, 1566, 1568–1570, 1595, 1613–1616, 1618–1619, 1621, 1623, 1625–1627, 1642, 1659–1660, 1662–1664, 1666–1667, 1669, 1671, 1673–1675, 1684–1687, 1696–1699, 1720–1723, 1744–1747, 1763–1766, 1846–1848, 1850–1851, 1853–1855, 1858–1861, 1863–1865, 1867–1871, 1909–1911, 1937–1982, 1993–1997
TOTAL	754	252	66%

Tests	Skipped	Failures	Errors	Time
108	0 💤	0 ❌	0 🔥	3.838s ⏱️

[marketplace-ai-reviewer]

🤖 AI Review Disclaimer

This review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause.

[marketplace-ai-reviewer]

Hi there, thanks for your contribution!

Overall the changes look good, but we noticed that the new build_connectivity_test_query function is missing unit tests. Please add test coverage in CortexDataLake_test.py to ensure the generated query includes the expected table, limit, and time bounds.

Once that's added, we should be good to proceed!

@TheL0L please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.

[content-bot]

Validate summary
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

[content-bot]

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 07041f1e5cad1592

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.