Skip to content

fix(rdf): four follow-up findings from Copilot review 4299978111

53a83b7
Select commit
Loading
Failed to load commit list.
Merged

fix(rdf): converge Fuseki state on weekly rebuilds and isolate API latency #28117

fix(rdf): four follow-up findings from Copilot review 4299978111
53a83b7
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 15, 2026 in 7m 27s

Annotations

Check warning on line 178 in src/components/common/MUIUserTeamSelect/MUIUserTeamSelect.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIdBF2iU0xy3YPmag&open=AZ4tIdBF2iU0xy3YPmag&pullRequest=28117

Check warning on line 40 in src/components/Learning/ResourcePlayer/StorylaneTour.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

'frameBorder' is deprecated.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIiXz2iU0xy3YPmgu&open=AZ4tIiXz2iU0xy3YPmgu&pullRequest=28117

Check warning on line 189 in src/utils/PipelineDetailsUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since the receiver accepts the original type of the expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIoaz2iU0xy3YPmpR&open=AZ4tIoaz2iU0xy3YPmpR&pullRequest=28117

Check warning on line 285 in src/components/Entity/EntityLineage/CustomNode.utils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Avoid non-native interactive elements. If using native HTML is not possible, add an appropriate role and support for tabbing, mouse, keyboard, and touch inputs to an interactive content element.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIhvv2iU0xy3YPmgG&open=AZ4tIhvv2iU0xy3YPmgG&pullRequest=28117

Check warning on line 85 in src/pages/TourPage/TourPage.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIl_x2iU0xy3YPmlf&open=AZ4tIl_x2iU0xy3YPmlf&pullRequest=28117

Check warning on line 246 in src/components/DataAssets/AssetsSelectionModal/useAssetSelectionContent.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since the receiver accepts the original type of the expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIim_2iU0xy3YPmg9&open=AZ4tIim_2iU0xy3YPmg9&pullRequest=28117

Check failure on line 281 in src/utils/BlockEditorUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `.dataset` over `setAttribute(…)`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tInlX2iU0xy3YPmoE&open=AZ4tInlX2iU0xy3YPmoE&pullRequest=28117

Check warning on line 397 in src/utils/CommonUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Use the "RegExp.exec()" method instead.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIob-2iU0xy3YPmpW&open=AZ4tIob-2iU0xy3YPmpW&pullRequest=28117

Check warning on line 522 in src/utils/GlossaryUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIpVp2iU0xy3YPmq5&open=AZ4tIpVp2iU0xy3YPmq5&pullRequest=28117

Check failure on line 269 in src/components/MyData/Widgets/KPIWidget/KPIWidget.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Provide a compare function to avoid sorting elements alphabetically.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIes12iU0xy3YPmcS&open=AZ4tIes12iU0xy3YPmcS&pullRequest=28117

Check warning on line 182 in src/components/KnowledgeCenter/KnowledgeDetailPageHeader/KnowledgeDetailPageHeader.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Move this component definition out of the parent component and pass data as props.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIhlM2iU0xy3YPmff&open=AZ4tIhlM2iU0xy3YPmff&pullRequest=28117

Check warning on line 103 in src/components/common/Table/TableV2Utils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIbba2iU0xy3YPmYO&open=AZ4tIbba2iU0xy3YPmYO&pullRequest=28117

Check warning on line 170 in src/components/ActivityFeed/ActivityFeedProvider/ActivityFeedProvider.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIfH12iU0xy3YPmcr&open=AZ4tIfH12iU0xy3YPmcr&pullRequest=28117

Check failure on line 436 in src/pages/ColumnBulkOperations/ColumnGrid/hooks/useColumnGridListingData.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 21 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIlmJ2iU0xy3YPmk8&open=AZ4tIlmJ2iU0xy3YPmk8&pullRequest=28117

Check warning on line 259 in src/pages/AddIngestionPage/AddIngestionPage.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since the receiver accepts the original type of the expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tImLo2iU0xy3YPmlu&open=AZ4tImLo2iU0xy3YPmlu&pullRequest=28117

Check warning on line 195 in src/hooks/useWorkflowState.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

`existingDataAssets` should be a `Set`, and use `existingDataAssets.has()` to check existence or non-existence.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIpsc2iU0xy3YPmrT&open=AZ4tIpsc2iU0xy3YPmrT&pullRequest=28117

Check warning on line 120 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using nullish coalescing operator (`??=`) instead of an assignment expression, as it is simpler to read.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIp1z2iU0xy3YPmrj&open=AZ4tIp1z2iU0xy3YPmrj&pullRequest=28117

Check warning on line 61 in src/components/ActivityFeed/ActivityFeedEditor/ActivityFeedEditorNew.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIe-Q2iU0xy3YPmci&open=AZ4tIe-Q2iU0xy3YPmci&pullRequest=28117

Check warning on line 31 in src/utils/Lineage/Layout/ELKUtil/ELKUtil.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Useless constructor.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tImbt2iU0xy3YPmmK&open=AZ4tImbt2iU0xy3YPmmK&pullRequest=28117

Check failure on line 464 in src/components/DataQuality/AddDataQualityTest/components/ParameterForm.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this code to not nest functions more than 4 levels deep.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tId0A2iU0xy3YPmbU&open=AZ4tId0A2iU0xy3YPmbU&pullRequest=28117

Check warning on line 171 in src/pages/EditConnectionFormPage/EditConnectionFormPage.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tImPv2iU0xy3YPml6&open=AZ4tImPv2iU0xy3YPml6&pullRequest=28117

Check failure on line 737 in src/components/DataContract/ODCSImportModal/ODCSImportModal.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIjOi2iU0xy3YPmho&open=AZ4tIjOi2iU0xy3YPmho&pullRequest=28117

Check warning on line 80 in src/utils/AlertsClassBase.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

The empty object is useless.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIpNW2iU0xy3YPmqp&open=AZ4tIpNW2iU0xy3YPmqp&pullRequest=28117

Check warning on line 133 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIp1z2iU0xy3YPmrm&open=AZ4tIp1z2iU0xy3YPmrm&pullRequest=28117

Check warning on line 48 in src/components/UploadFile/UploadFile.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Blob#text()` over `FileReader#readAsText(blob)`.

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ4tIkoi2iU0xy3YPmjq&open=AZ4tIkoi2iU0xy3YPmjq&pullRequest=28117