[Michael Flanakin (flanakin)]

🛠️ Description

Initial thoughts about a FOCUS query library.

📋 Checklist

🔬 How did you test this change?

• 🤏 Lint tests
• 🤞 PS -WhatIf / az validate
• 👍 Manually deployed + verified
• 💪 Unit tests
• 🙌 Integration tests

🙋‍♀️ Do any of the following that apply?

• 🚨 This is a breaking change.
• 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

• ✅ Updated changelog (required for dev PRs)
• ➡️ Will add log in a future PR (feature branch PRs only)
• ❎ Log not needed (small/internal change)

📖 Did you update documentation?

• ✅ Public docs in docs (required for dev)
• ✅ Internal dev docs in src (required for dev)
• ➡️ Will add docs in a future PR (feature branch PRs only)
• ❎ Docs not needed (small/internal change)

[Roland Krummenacher (RolandKrummenacher)]

Shall we avoid overlaps with the FinOps Foundation's Use Case Library? Will that library be populated with Frequently Asked Queries?

[Michael Flanakin (flanakin)]

🤖 [AI] ❓ Question

Overlap with the FinOps Foundation Use Case Library is a scope call for the PR author; flagged for decision.

[Roland Krummenacher (RolandKrummenacher)]

The KQL change in 6760d14 is fine. The SQL example doesn't run, and both queries mix currencies.

On the Use Case Library thread: your AI flagged it for your decision, so it stays with you.

[microsoft-github-policy-service]

@Michael Flanakin (@flanakin): you have some new feedback!

Please review and resolve all comments and I'll let reviewers know by removing the Needs: Attention label. If I miss anything, just reply with #needs-review and I'll update the status.

[Roland Krummenacher (RolandKrummenacher)]

Thanks, d82b194 fixes all three points: the SQL now groups by every selected column, the cost column has an alias, and both queries split by BillingCurrency. The queries are consistent with each other and the SQL is valid T-SQL. Lifting my change request.

The Use Case Library overlap question is still open with you. It's your call and doesn't block this PR.

Non-blocking: focus/queries.md isn't in docs-mslearn/TOC.yml yet, so the page won't show up in the Learn navigation. Add it under the FOCUS section unless it's meant to stay unlisted for now.