yujunjun/datahub
2
0
Fork 0
mirror of https://github.com/datahub-project/datahub.git synced 2025-09-03 06:13:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(backend): do not lower-case dataset key parts for browse paths (#10385)

Browse Source
This commit is contained in:
ksrinath 2024-04-29 19:33:29 +05:30 committed by GitHub
parent 1af050c730
commit 4c58feb733
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
metadata-io/src/main/java/com/linkedin/metadata/search/utils/BrowsePathV2Utils.java
View File

@ -67,8 +67,7 @@ public class BrowsePathV2Utils {
BrowsePathEntryArray defaultDatasetPathEntries = BrowsePathEntryArray defaultDatasetPathEntries =
getDefaultDatasetPathEntries(dsKey.getName(), dataPlatformDelimiter); getDefaultDatasetPathEntries(dsKey.getName(), dataPlatformDelimiter);
if (defaultDatasetPathEntries.size() > 0) { if (defaultDatasetPathEntries.size() > 0) {
browsePathEntries.addAll( browsePathEntries.addAll(defaultDatasetPathEntries);
getDefaultDatasetPathEntries(dsKey.getName().toLowerCase(), dataPlatformDelimiter));
} else { } else {
browsePathEntries.add(createBrowsePathEntry(DEFAULT_FOLDER_NAME, null)); browsePathEntries.add(createBrowsePathEntry(DEFAULT_FOLDER_NAME, null));
} }

4
metadata-io/src/test/java/com/linkedin/metadata/search/utils/BrowsePathV2UtilsTest.java
View File

@ -146,8 +146,8 @@ public class BrowsePathV2UtilsTest {
BrowsePathV2Utils.getDefaultBrowsePathV2( BrowsePathV2Utils.getDefaultBrowsePathV2(
mock(OperationContext.class), datasetUrn, this.registry, '.', mockService, true); mock(OperationContext.class), datasetUrn, this.registry, '.', mockService, true);
BrowsePathEntryArray expectedPath = new BrowsePathEntryArray(); BrowsePathEntryArray expectedPath = new BrowsePathEntryArray();
BrowsePathEntry entry1 = new BrowsePathEntry().setId("test"); BrowsePathEntry entry1 = new BrowsePathEntry().setId("Test");
BrowsePathEntry entry2 = new BrowsePathEntry().setId("a"); BrowsePathEntry entry2 = new BrowsePathEntry().setId("A");
expectedPath.add(entry1); expectedPath.add(entry1);
expectedPath.add(entry2); expectedPath.add(entry2);
Assert.assertEquals(browsePathsV2.getPath(), expectedPath); Assert.assertEquals(browsePathsV2.getPath(), expectedPath);