Skip to content

Repo sync for protected branch #2528

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
learn-build-service-prod merged 5 commits into from
Jan 28, 2025
Merged

Repo sync for protected branch #2528

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
6b74ac3
remove wrong null counting note
michaelbrichko Jan 26, 2025
5d88826
Merge pull request #6573 from michaelbrichko/user/mbrichko/aggregatio…
prmerger-automator[bot] Jan 28, 2025
667dc1d
Update pull_request_template.md
shsagir Jan 28, 2025
d0d7201
Merge pull request #6591 from MicrosoftDocs/main
v-alje Jan 28, 2025
148b6aa
Merging changes synced from https://github.com/MicrosoftDocs/dataexpl…
Jan 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/pull_request_template.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ Please add a brief comment outlining the purpose of this PR. Add links to any re

1. **Acrolinx:** Make sure your Acrolinx score is **at least 80** (higher is better) and with **0** spelling issues.
1. **Successful build**: Review the build status to make sure **all files are green** (Succeeded) and there are no errors, warnings, or suggestions.
1. **Preview the pages:**: Click each **Preview URL** link, scan the entire page looking for formatting issues, in particular the parts you edited.
1. **Check the Table of Contents:** If you are adding a new markdown file, make sure it is linked from the table of contents.
1. **Preview the pages**: Click each **Preview URL** link, scan the entire page looking for formatting issues, in particular the parts you edited.
1. **Check the Table of Contents:** If you're adding a new markdown file, make sure it is linked from the table of contents.
1. **Sign off**: Once the PR is finalized, add a comment with `#sign-off` . If you need to cancel the sign-off, add a comment with `#hold-off`.

**NOTE**: *Signing off means the document can be published at any time.*
Expand All @@ -17,4 +17,4 @@ Please add a brief comment outlining the purpose of this PR. Add links to any re
- All PRs to this repository are reviewed and merged by a human. Automatic merge is disabled on this repository for PRs, even with the qualifies-for-auto-merge label.
- Once all feedback on the PR is addressed, the PR will be merged into the main branch.

[Learn more about how to contribute](https://review.learn.microsoft.com/en-us/help/platform/?branch=main).
[Learn more about how to contribute](https://review.learn.microsoft.com/en-us/help/platform/?branch=main)
2 changes: 0 additions & 2 deletions data-explorer/kusto/query/count-aggregation-function.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,8 @@ monikerRange: "microsoft-fabric || azure-data-explorer || azure-monitor || micro

> [!INCLUDE [applies](../includes/applies-to-version/applies.md)] [!INCLUDE [fabric](../includes/applies-to-version/fabric.md)] [!INCLUDE [azure-data-explorer](../includes/applies-to-version/azure-data-explorer.md)] [!INCLUDE [monitor](../includes/applies-to-version/monitor.md)] [!INCLUDE [sentinel](../includes/applies-to-version/sentinel.md)]


Counts the number of records per summarization group, or total if summarization is done without grouping.

[!INCLUDE [ignore-nulls](../includes/ignore-nulls.md)]
::: moniker range="microsoft-fabric || azure-data-explorer || azure-monitor || microsoft-sentinel"
To only count records for which a predicate returns `true`, use [countif()](countif-aggregation-function.md).
::: moniker-end
Expand Down
2 changes: 0 additions & 2 deletions data-explorer/kusto/query/count-distinct-aggregation-function.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ ms.date: 08/11/2024

Counts unique values specified by the scalar expression per summary group, or the total number of unique values if the summary group is omitted.

[!INCLUDE [ignore-nulls](../includes/ignore-nulls.md)]

[!INCLUDE [data-explorer-agg-function-summarize-note](../includes/agg-function-summarize-note.md)]

If you only need an estimation of unique values count, we recommend using the less resource-consuming [`dcount`](dcount-aggregation-function.md) aggregation function.
Expand Down
2 changes: 0 additions & 2 deletions data-explorer/kusto/query/countif-aggregation-function.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ ms.date: 08/11/2024

Counts the rows in which *predicate* evaluates to `true`.

[!INCLUDE [ignore-nulls](../includes/ignore-nulls.md)]

[!INCLUDE [data-explorer-agg-function-summarize-note](../includes/agg-function-summarize-note.md)]

## Syntax
Expand Down
3 changes: 0 additions & 3 deletions data-explorer/kusto/query/scalar-data-types/null-values.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,6 @@ datatable(val:int)[5, int(null)]

When applying the following operators to entities that include null values, the null values are ignored and don't factor into the calculation:

* [count()](../count-aggregation-function.md)
* [count_distinct()](../count-distinct-aggregation-function.md)
* [countif()](../countif-aggregation-function.md)
* [dcount()](../dcount-aggregation-function.md)
* [dcountif()](../dcountif-aggregation-function.md)
* [make_bag()](../make-bag-aggregation-function.md)
Expand Down