You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/03/13 15:34:03 UTC
[GitHub] [arrow-datafusion] Igosuki opened a new pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Igosuki opened a new pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003
# Which issue does this PR close?
.
# Rationale for this change
Stabilize the branch around arrow2 0.10, which has fixes for nested data structs
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] houqp commented on pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
houqp commented on pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1074832589
FYI, I also pinned `parquet-format-async-temp` version to 0.2 to unblock the build.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] houqp commented on pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
houqp commented on pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1068770842
Thanks @Igosuki I am going to try finding time going over it this week. Also working on a local fix for the json fileformat plan at the moment and will rebase my fix once this PR is fixed.
I think the commit still looks wrong to me: master commits got merged into a single commit https://github.com/apache/arrow-datafusion/pull/2003/commits/cbaf9db8e2cb40f69cb532b71206eec47a119d7d. I will help take a look into how to better unsquash these commits after this PR is merged as well.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] houqp closed pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
houqp closed pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] houqp commented on pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
houqp commented on pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1074831772
@Igosuki I found a way squashed all the arrow2 related commits into a single one so we can unsquash all new master commits in the arrow2 branch so we can start using 3 way merge going forward to reduce merge conflicts.
The arrow2 branch has been force updated including the changes you have in this PR. I will send the json test fix later this week to the arrow2 branch and let you know.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] houqp edited a comment on pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
houqp edited a comment on pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1074831772
@Igosuki I found a way to squash all the arrow2 related commits into a single one so we can unsquash all new master commits in the arrow2 branch. With a clean history, we can now start using 3 way merge going forward to reduce merge conflicts.
The arrow2 branch has been force updated including the changes you have in this PR. I will send the json test fix later this week to the arrow2 branch and let you know.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] Igosuki commented on pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
Igosuki commented on pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1074892819
Nice !
On Tue, Mar 22, 2022 at 8:38 AM QP Hou ***@***.***> wrote:
> FYI, I also pinned parquet-format-async-temp version to 0.2 to unblock
> the build.
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1074832589>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AADDFBRMG22YSBJLWVKFMB3VBF2INANCNFSM5QTPR5UA>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] Igosuki commented on pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
Igosuki commented on pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1068968018
I should probably have used the new three way merge you talked about
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] houqp commented on pull request #2003: Merges master and integrates arrow2/parquet2 0.10
Posted by GitBox <gi...@apache.org>.
houqp commented on pull request #2003:
URL: https://github.com/apache/arrow-datafusion/pull/2003#issuecomment-1070321256
no worries, we can give it a try next time we catch up to master :)
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org