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

Using Snowflake timestamp when casted can create issues while adding KPI #320

Closed
suranah opened this issue Oct 14, 2021 · 3 comments
Closed
Assignees
Labels
🔗 connectors DB, DW & third party connectors P1

Comments

@suranah
Copy link
Contributor

suranah commented Oct 14, 2021

Describe the bug
Using Snowflake timestamp when cast from another column is creating issues while adding KPI.

Explain the environment

  • Chaos Genius version: example is 0.1.2-alpha
  • OS Version / Instance: TBA
  • Deployment type: Docker

Current behavior
It returns an error dialog with the message - Error: 'timestamp column'. This is currently observed for a custom query.

Expected behavior
If the KPI definition follows all validation checks, it should be added as a KPI and subsequent analytics processed on it.

@suranah suranah added the 🔗 connectors DB, DW & third party connectors label Oct 14, 2021
@suranah suranah added the P1 label Oct 14, 2021
@kartikay-bagla
Copy link
Contributor

This looks like an issue with query based KPIs only on Snowflake. We again tested it well with table based KPIs. This seems to be an issue with our metadata ingester. Will update on sometime for this.

@suranah
Copy link
Contributor Author

suranah commented Oct 18, 2021

@manassolanki do update on the fixes here for @gxu-kangaroo once you are done.

@manassolanki
Copy link
Member

@gxu-kangaroo Snowflake metadata is case insensitive and snowflake DBAPI converts the metadata info to the uppercase. This is causing the issue while we are trying to match KPI definition (defined via metadata) with the query result. We have replaced the DBAPI cursor with the SqlAlchemy ResultProxy (which is wrapper around the DBAPI cursor to make things consistent across various DB/DW)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔗 connectors DB, DW & third party connectors P1
Projects
None yet
Development

No branches or pull requests

3 participants