Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Consider switching all self-describing columns to VARIANT type #64

Open
chuwy opened this issue Oct 24, 2018 · 1 comment
Open

Consider switching all self-describing columns to VARIANT type #64

chuwy opened this issue Oct 24, 2018 · 1 comment
Assignees

Comments

@chuwy
Copy link
Contributor

chuwy commented Oct 24, 2018

When we were designing initial version of Snowflake Loader, we made a decision to leave all contexts as ARRAY type (because we always have zero or more of them, always wrapped in array) and OBJECT for unstructured events (because it is a JSON-object).

From my understanding ARRAY and OBJECT are both subtypes of VARIANT. However we're having a user who claims that OBJECT type breaks an escaping (for |-symbol), whereas it works for VARIANT. From my point of view, this is an unexpected behavior and should not happen.

We're investigating this with Snowflake engineers, but if it will turn out that this is an expected behavior then we'll need to change type of all self-describing types to VARIANT. Adding to 0.4.0 so far, but probably it won't be necessary.

@chuwy chuwy added this to the Version 0.4.0 milestone Oct 24, 2018
@chuwy
Copy link
Contributor Author

chuwy commented Oct 29, 2018

This behavior was confirmed as a bug in Snowflake/Databricks integration. Closing this.

@chuwy chuwy closed this as completed Oct 29, 2018
@chuwy chuwy removed this from the Version 0.4.0 milestone Dec 10, 2018
@chuwy chuwy reopened this Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants