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

Expand Namespace character support #1839

Closed
TohnJhomas opened this issue Jan 18, 2022 · 3 comments · Fixed by #1972
Closed

Expand Namespace character support #1839

TohnJhomas opened this issue Jan 18, 2022 · 3 comments · Fixed by #1972
Labels
bug Something isn't working
Milestone

Comments

@TohnJhomas
Copy link

TohnJhomas commented Jan 18, 2022

A spark integration is automatically creating the namespace abfss://root@datalakedev.dfs.core.windows.net for a user, despite the @ symbol not being supported in namespaces.

As a result, the api is sending 400 errors in response to runEvents.

We should look into expanding to support a broader character set in order to head off these issues in the future. In the immediate sense, we at least need to support the @ symbol to get this user unstuck.

This user has created an alias in the interim, but aren't able to fully resolve the problem without editing the spark integration

@TohnJhomas TohnJhomas changed the title Expand supported Namespace characters Expand Namespace character support Jan 18, 2022
@darezik
Copy link

darezik commented Jan 18, 2022

This also appears to be the root cause for #1793

@wslulciuc wslulciuc added the bug Something isn't working label Mar 31, 2022
@wslulciuc
Copy link
Member

Fixed #1925

@wslulciuc wslulciuc moved this to Done in Marquez Mar 31, 2022
@wslulciuc wslulciuc added this to the 0.22.0 milestone Mar 31, 2022
@gage-russell
Copy link

@wslulciuc Why was this closed?

#1925 addresses maximum length of 1024 characters.

But the issue is that @ is not included in the regex check for namespace.

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
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants