MeltanoLabs / target-athena

Singer.io Target for AWS Athena.
Other
5 stars 16 forks source link

Remove table name hyphens #42

Closed pnadolny13 closed 2 years ago

pnadolny13 commented 2 years ago

Problem

Describe the problem your PR is trying to solve

Proposed changes

Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.

Related to https://github.com/MeltanoLabs/target-athena/issues/33. Tables fail to get created when the stream name include hyphens which are not allowed.

Types of changes

What types of changes does your code introduce to PipelineWise? Put an x in the boxes that apply

Checklist

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

aaronsteers commented 2 years ago

@pnadolny13 - By chance do you have time to log this as an issue in the SDK? Historically, the dash has been used as a delimiter between table, scheme, and db name parts.

pnadolny13 commented 2 years ago

By chance do you have time to log this as an issue in the SDK?

@aaronsteers Done! https://gitlab.com/meltano/sdk/-/issues/344

aaronsteers commented 2 years ago

Thanks, @pnadolny13 !