You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In ion-hive-serde, we are using an ion struct to store values fetched from a path extractor because of:
We need a key-value store to carry column values
The top-level IonStruct as a context object carries the IonSystem which we use as a ValueFactory in the callbacks created in PathExtractionConfig
As mentioned in above comment,
Motivation for using a non-IonValue as our top level key-value store and context object is that it intentionally disrupts handling of recursive IonValues. This shouldn't be an IonStruct- it's a collection of columns whose values are IonValues.
It would be good to replace our path extractor context object with a Map<String,IonValue> or some similar non-IonValue context object.
The text was updated successfully, but these errors were encountered:
This issue was raised in #59 (comment)
In ion-hive-serde, we are using an ion struct to store values fetched from a path extractor because of:
As mentioned in above comment,
It would be good to replace our path extractor context object with a Map<String,IonValue> or some similar non-IonValue context object.
The text was updated successfully, but these errors were encountered: