timsaucer merged PR #1085:
URL: https://github.com/apache/datafusion-python/pull/1085
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...
timsaucer commented on PR #1085:
URL:
https://github.com/apache/datafusion-python/pull/1085#issuecomment-2833467666
I'm sorry it took me so long to get around to reviewing this. Thank you for
the contribution!
--
This is an automated message from the Apache Git Service.
To respond to the
chenkovsky commented on PR #1085:
URL:
https://github.com/apache/datafusion-python/pull/1085#issuecomment-2764593959
> This is a really big PR and it's not immediately obvious to me what
problem it's trying to solve. Can you expand the description or at least link
it to an issue describing
timsaucer commented on PR #1085:
URL:
https://github.com/apache/datafusion-python/pull/1085#issuecomment-2764531402
This is a really big PR and it's not immediately obvious what problem it's
trying to solve. Can you expand the description or at least link it to an issue
describing the prob
chenkovsky opened a new pull request, #1085:
URL: https://github.com/apache/datafusion-python/pull/1085
# Which issue does this PR close?
# Rationale for this change
Some LogicalPlans don't support to_variant now.
# What changes are included in this PR?
add python
