Skip to content
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.

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

Purview ADLS Gen1 mount to URL resolve mismatch #175

Closed
Kishor-Radhakrishnan opened this issue Feb 27, 2023 · 4 comments
Closed

Purview ADLS Gen1 mount to URL resolve mismatch #175

Kishor-Radhakrishnan opened this issue Feb 27, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@Kishor-Radhakrishnan
Copy link

Kishor-Radhakrishnan commented Feb 27, 2023

Describe the bug

We are trying out latest version of ADB lineage solution. And we are using mounts in our notebook. But ADLS URL resolved in purview while using this solution is not matching with actual URL.

URL in purview from solution : adl://accountname.azuredatalakestore.net/lob/ontology/path1/path2/path3

expected actual ADLS Url : adl://accountname.azuredatalakestore.net/datalake-dev/lob/ontology/path1/path2/path3

Mount in Databricks :

MountPoint - /mnt/datalake/
Source - adl://accountname.azuredatalakestore.net/datalake-dev/

Path used in notebook ( as mount) - /mnt/datalake/lob/ontology/path1/path2/path3

@Kishor-Radhakrishnan Kishor-Radhakrishnan added the bug Something isn't working label Feb 27, 2023
@wjohnson
Copy link
Collaborator

wjohnson commented Mar 1, 2023

Thank you @Kishor-Radhakrishnan for identifying this issue!

I created this hotfix branch to hopefully resolve the issue. It will require you go redeploy the function and update the OlToPurviewMappings app setting.

New Branch: https://github.com/microsoft/Purview-ADB-Lineage-Solution-Accelerator/tree/hotfix/mntDepth
Updated ADLG1 Mapping: https://github.com/microsoft/Purview-ADB-Lineage-Solution-Accelerator/blob/hotfix/mntDepth/docs/mappings/adlsg1.json

If you would be able to test this fix we can add this hotfix and your other issue to the latest releases.

@Kishor-Radhakrishnan
Copy link
Author

Kishor-Radhakrishnan commented Mar 1, 2023

@wjohnson Tested it out , Paths resolving properly now . Thanks for the fix again.

Screenshot 2023-03-01 at 7 37 01 PM

@wjohnson
Copy link
Collaborator

wjohnson commented Mar 2, 2023

@Kishor-Radhakrishnan thank you for testing! We will get this PR into the release branch as quickly as possible!

@wjohnson
Copy link
Collaborator

Closing as this made it into main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants