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

…atahub-project#10385)
darnaut · Apr 29, 2024 · 4c58feb · 4c58feb
1 parent 1af050c
commit 4c58feb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/metadata-io/src/main/java/com/linkedin/metadata/search/utils/BrowsePathV2Utils.java b/metadata-io/src/main/java/com/linkedin/metadata/search/utils/BrowsePathV2Utils.java
@@ -67,8 +67,7 @@ public static BrowsePathsV2 getDefaultBrowsePathV2(
  BrowsePathEntryArray defaultDatasetPathEntries =
  getDefaultDatasetPathEntries(dsKey.getName(), dataPlatformDelimiter);
  if (defaultDatasetPathEntries.size() > 0) {
- browsePathEntries.addAll(
- getDefaultDatasetPathEntries(dsKey.getName().toLowerCase(), dataPlatformDelimiter));
+ browsePathEntries.addAll(defaultDatasetPathEntries);
  } else {
  browsePathEntries.add(createBrowsePathEntry(DEFAULT_FOLDER_NAME, null));
  }

diff --git a/metadata-io/src/test/java/com/linkedin/metadata/search/utils/BrowsePathV2UtilsTest.java b/metadata-io/src/test/java/com/linkedin/metadata/search/utils/BrowsePathV2UtilsTest.java
@@ -146,8 +146,8 @@ public void testGetDefaultBrowsePathV2WithoutContainers() throws URISyntaxExcept
  BrowsePathV2Utils.getDefaultBrowsePathV2(
  mock(OperationContext.class), datasetUrn, this.registry, '.', mockService, true);
  BrowsePathEntryArray expectedPath = new BrowsePathEntryArray();
- BrowsePathEntry entry1 = new BrowsePathEntry().setId("test");
- BrowsePathEntry entry2 = new BrowsePathEntry().setId("a");
+ BrowsePathEntry entry1 = new BrowsePathEntry().setId("Test");
+ BrowsePathEntry entry2 = new BrowsePathEntry().setId("A");
  expectedPath.add(entry1);
  expectedPath.add(entry2);
  Assert.assertEquals(browsePathsV2.getPath(), expectedPath);