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

namespaces: allow '=' and ';' in names #1936

Merged
merged 1 commit into from Apr 7, 2022

Conversation

mobuchowski
Copy link
Contributor

Solve bug in issue OpenLineage/OpenLineage#655 by allowing ';' and '=' in namespace names.

Signed-off-by: Maciej Obuchowski obuchowski.maciej@gmail.com

@collado-mike collado-mike enabled auto-merge (squash) April 6, 2022 15:49
@mobuchowski mobuchowski force-pushed the bug/allow-additional-namespace-chars branch from a11780e to c132f22 Compare April 6, 2022 16:10
Copy link
Member

@wslulciuc wslulciuc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@mobuchowski mobuchowski closed this Apr 7, 2022
auto-merge was automatically disabled April 7, 2022 12:39

Pull request was closed

@mobuchowski mobuchowski force-pushed the bug/allow-additional-namespace-chars branch from c132f22 to b583633 Compare April 7, 2022 12:39
@mobuchowski mobuchowski deleted the bug/allow-additional-namespace-chars branch April 7, 2022 12:40
@mobuchowski mobuchowski restored the bug/allow-additional-namespace-chars branch April 7, 2022 12:40
@mobuchowski
Copy link
Contributor Author

Accidentally closed, reopening

@mobuchowski mobuchowski reopened this Apr 7, 2022
@mobuchowski mobuchowski force-pushed the bug/allow-additional-namespace-chars branch from feee49c to 01590d1 Compare April 7, 2022 13:10
Signed-off-by: Maciej Obuchowski <obuchowski.maciej@gmail.com>
@mobuchowski mobuchowski force-pushed the bug/allow-additional-namespace-chars branch from 01590d1 to 0696026 Compare April 7, 2022 13:36
@codecov
Copy link

codecov bot commented Apr 7, 2022

Codecov Report

Merging #1936 (0696026) into main (b583633) will increase coverage by 13.49%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##               main    #1936       +/-   ##
=============================================
+ Coverage     64.59%   78.09%   +13.49%     
- Complexity      184      944      +760     
=============================================
  Files            35      193      +158     
  Lines           771     5249     +4478     
  Branches         86      418      +332     
=============================================
+ Hits            498     4099     +3601     
- Misses          146      705      +559     
- Partials        127      445      +318     
Impacted Files Coverage Δ
...main/java/marquez/common/models/NamespaceName.java 80.00% <100.00%> (ø)
api/src/main/java/marquez/api/RunResource.java 86.66% <0.00%> (ø)
...rc/main/java/marquez/tracing/TracingSQLLogger.java 0.00% <0.00%> (ø)
...in/java/marquez/common/base/MorePreconditions.java 100.00% <0.00%> (ø)
.../src/main/java/marquez/service/DelegatingDaos.java 0.00% <0.00%> (ø)
.../src/main/java/marquez/service/models/Lineage.java 80.00% <0.00%> (ø)
api/src/main/java/marquez/api/JobResource.java 97.82% <0.00%> (ø)
.../src/main/java/marquez/service/DatasetService.java 89.18% <0.00%> (ø)
...ain/java/marquez/graphql/GraphqlSchemaBuilder.java 88.80% <0.00%> (ø)
...pi/src/main/java/marquez/tracing/SentryConfig.java 100.00% <0.00%> (ø)
... and 149 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@mobuchowski mobuchowski merged commit ef063d7 into main Apr 7, 2022
@mobuchowski mobuchowski deleted the bug/allow-additional-namespace-chars branch April 7, 2022 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants