Skip to content

Rows#103

Merged
rcypher-databricks merged 6 commits into
databricks:arrowfrom
rcypher-databricks:rows
Feb 14, 2023
Merged

Rows#103
rcypher-databricks merged 6 commits into
databricks:arrowfrom
rcypher-databricks:rows

Conversation

@rcypher-databricks

@rcypher-databricks rcypher-databricks commented Feb 14, 2023

Copy link
Copy Markdown
Contributor

Support for arrow format results.
Includes support for timestamp and complex types returned using arrow native types.

Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
Removed the withArrowBatches function and changed default configuration to use arrow batches.
Fixed error message in arrowRows.
Updated test that wasn't checking for valid json when handling complex types.
Added comments w.r.t. the logger in the arrowRowScanner struct.

Signed-off-by: Raymond Cypher <raymond.cypher@databricks.com>
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.

1 participant