Skip to content

NAA SQL DC Schema documentation - v11.6#413

Merged
tay-caliguiri merged 5 commits intodevfrom
naa-sql-schema-doc
Mar 6, 2026
Merged

NAA SQL DC Schema documentation - v11.6#413
tay-caliguiri merged 5 commits intodevfrom
naa-sql-schema-doc

Conversation

@tay-caliguiri
Copy link
Contributor

  • Added NAA v11.6\Administration\Schema directory and appropriate subdirectories
  • Re-formatted the NAA_SQL_DC_Schema_Documentation_11.6.md from dev/@sanjay-meghani for the doc site and easy navigation
  • Minimally fixed phrasing on the SQL Schema Overview page

…directories

- Re-formatted the NAA_SQL_DC_Schema_Documentation_11.6.md from dev/@sanjay-meghani for the doc site and easy navigation
- Minimally fixed phrasing on the SQL Schema Overview page
@tay-caliguiri tay-caliguiri requested a review from a team as a code owner March 5, 2026 23:23
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

None.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

Correction : 59 tables instead of 56
Correction : 59 tables instead of 56
Correction : 59 tables instead of 56
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/accessanalyzer/11.6/admin/schema/overview.md

Line 3: Impersonal construction "This section provides" should be rewritten in active voice. Replace with direct statement like "Access Analyzer 11.6 data collectors store discovered configuration, permissions, and audit data in normalized tables, with denormalized reporting views for analysis and presentation."

docs/accessanalyzer/11.6/admin/schema/sqldc/overview.md

Line 5: Weak construction "This is a comprehensive schema documentation" should be rewritten. Remove "This is" and make it direct. Also, "NAA" appears to be undefined—spell out on first use or use "Access Analyzer" or the full product name.

Line 5: Passive construction "The SQL Server Data Collector is a component of the NAA 11.6 platform that discovers" should be rewritten in active voice. Rephrase to focus on what the collector does, not what it "is."

Line 11: Sentence fragment "Includes shared infrastructure tables" following a period should either be incorporated into the previous sentence or made into a complete sentence.

docs/accessanalyzer/11.6/admin/schema/sqldc/enumeration/overview.md

Line 3: Impersonal construction "This section documents" should be rewritten without "This section."

Line 9: Passive voice "Referenced by" appears repeatedly throughout this file as the opening phrase for multiple sections. Rewrite in active voice or restructure to avoid passive construction.

Lines 22, 42, 57, 88: Impersonal construction "Actual values present in the database depend on" should be rewritten. Rephrase to be more direct without the impersonal subject.

Lines 9, 28, 48, 64, 94: Passive voice "Values are populated at runtime" appears multiple times. Rewrite in active voice to indicate who/what populates the values.

docs/accessanalyzer/11.6/admin/schema/sqldc/erd/overview.md

Line 5: Passive voice "The schema is organized into four major subsystems" should be rewritten in active voice.

Line 5: Impersonal construction "This overview shows" should be rewritten without "This overview."

docs/accessanalyzer/11.6/admin/schema/sqldc/reportingtables/overview.md

Line 3: Passive voice "The following 59 tables are denormalized reporting tables populated by NAA analysis jobs" should be rewritten. Replace passive "are...populated" with active construction.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/accessanalyzer/11.6/admin/schema/sqldc/overview.md

Line 11: Passive voice. Change "Normalized tables populated directly by data collectors" to "Data collectors populate these normalized tables directly".

Line 13: Passive voice. Change "Denormalized, report-ready tables populated by analysis jobs" to "Analysis jobs populate these denormalized, report-ready tables".

docs/accessanalyzer/11.6/admin/schema/sqldc/enumeration/overview.md

Line 9: Passive voice. Change "Values are populated at runtime from" to "The system populates values at runtime from".

Line 28: Passive voice. Change "Values are populated at runtime from" to "The system populates values at runtime from".

Line 48: Passive voice. Change "Values are populated at runtime from" to "The system populates values at runtime from".

Line 64: Passive voice. Change "Values are populated at runtime from" to "The system populates values at runtime from".

Line 94: Passive voice. Change "Values are populated at runtime from" to "The system populates values at runtime from". Also change "additional permissions may be present" to "additional permissions can exist".

Line 124: Passive voice. Change "the most commonly encountered types are shown below" to "the table below shows the most commonly encountered types".

Line 145: Passive voice. Change "107 class types are defined" to "The system defines 107 class types".

Line 151: Passive voice. Change "frequently referenced actions are shown below" to "the table below shows frequently referenced actions".

docs/accessanalyzer/11.6/admin/schema/sqldc/functions/overview.md

Line 36: Jargon. Replace "Upserts" with "Inserts or updates" for clarity.

docs/accessanalyzer/11.6/admin/schema/sqldc/reportingtables/overview.md

Line 3: Passive voice. Change "denormalized reporting tables populated by NAA analysis jobs" to "denormalized reporting tables that NAA analysis jobs populate". Also change "are designed for direct consumption by" to "provide data for".

docs/accessanalyzer/11.6/admin/schema/overview.md

None.

docs/accessanalyzer/11.6/admin/schema/sqldc/coretables/overview.md

None.

docs/accessanalyzer/11.6/admin/schema/sqldc/erd/overview.md

None.

docs/accessanalyzer/11.6/admin/schema/sqldc/indexreference/overview.md

None.

docs/accessanalyzer/11.6/admin/schema/sqldc/views/overview.md

None.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/accessanalyzer/11.6/admin/schema/sqldc/overview.md

Line 11: Passive voice "Normalized tables populated directly by data collectors" — rewrite in active voice with data collectors as the subject.

Line 13: Passive voice "Denormalized, report-ready tables populated by analysis jobs" — rewrite in active voice with analysis jobs as the subject.

docs/accessanalyzer/11.6/admin/schema/sqldc/reportingtables/overview.md

Line 3: Passive voice "denormalized reporting tables populated by NAA analysis jobs" — rewrite in active voice.

Line 3: Passive voice "are designed for direct consumption by the NAA console and reports" — rewrite in active voice.

Line 15: Passive voice "database activity events captured from SQL Server audit logs" — rewrite in active voice.

Line 15: Passive voice "a single audited action performed by a user" — rewrite in active voice.

docs/accessanalyzer/11.6/admin/schema/sqldc/enumeration/overview.md

Line 9: Passive voice "Referenced by" — rewrite with the referencing column as the subject. Also "Values are populated" is passive — rewrite in active voice.

Line 28: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 48: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 64: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 94: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 124: Passive voice "Referenced by" — rewrite in active voice.

Line 151: Passive voice "Referenced by" — rewrite in active voice.

Line 211: Passive voice "Referenced by" — rewrite in active voice.

Line 226: Passive voice "Used in" — rewrite in active voice.

Line 256: Passive voice "Used to compute" — rewrite in active voice.

Line 145: Passive voice "107 class types are defined" — rewrite in active voice.

Line 124: Passive voice "are shown below" — rewrite in active voice.

Line 151: Passive voice "are shown below" — rewrite in active voice.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/accessanalyzer/11.6/admin/schema/sqldc/overview.md

Line 11: Passive voice "Normalized tables populated directly by data collectors" — rewrite in active voice with data collectors as the subject.

Line 13: Passive voice "Denormalized, report-ready tables populated by analysis jobs" — rewrite in active voice with analysis jobs as the subject.

docs/accessanalyzer/11.6/admin/schema/sqldc/reportingtables/overview.md

Line 3: Passive voice "denormalized reporting tables populated by NAA analysis jobs" — rewrite in active voice.

Line 3: Passive voice "are designed for direct consumption by the NAA console and reports" — rewrite in active voice.

Line 15: Passive voice "database activity events captured from SQL Server audit logs" — rewrite in active voice.

Line 15: Passive voice "a single audited action performed by a user" — rewrite in active voice.

docs/accessanalyzer/11.6/admin/schema/sqldc/enumeration/overview.md

Line 9: Passive voice "Referenced by" — rewrite with the referencing column as the subject. Also "Values are populated" is passive — rewrite in active voice.

Line 28: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 48: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 64: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 94: Passive voice "Referenced by" and "Values are populated" — rewrite in active voice.

Line 124: Passive voice "Referenced by" — rewrite in active voice.

Line 151: Passive voice "Referenced by" — rewrite in active voice.

Line 211: Passive voice "Referenced by" — rewrite in active voice.

Line 226: Passive voice "Used in" — rewrite in active voice.

Line 256: Passive voice "Used to compute" — rewrite in active voice.

Line 145: Passive voice "107 class types are defined" — rewrite in active voice.

Line 124: Passive voice "are shown below" — rewrite in active voice.

Line 151: Passive voice "are shown below" — rewrite in active voice.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/accessanalyzer/11.6/admin/schema/sqldc/overview.md

Line 5: NAA acronym used without spelling it out on first use. Spell out as "Netwrix Access Analyzer (NAA)".

docs/accessanalyzer/11.6/admin/schema/sqldc/reportingtables/overview.md

Line 3: NAA acronym used without spelling it out on first use. Spell out as "Netwrix Access Analyzer (NAA)".

docs/accessanalyzer/11.6/admin/schema/sqldc/functions/overview.md

Line 18: SID acronym used without spelling it out on first use. Spell out as "Security Identifier (SID)".

docs/accessanalyzer/11.6/admin/schema/sqldc/views/overview.md

Line 37: SID acronym used without spelling it out on first use in column description. Spell out as "Security Identifier (SID)".


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

@tay-caliguiri tay-caliguiri merged commit 8915c4b into dev Mar 6, 2026
8 checks passed
@tay-caliguiri tay-caliguiri deleted the naa-sql-schema-doc branch March 6, 2026 18:01
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.

2 participants