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/09/27 12:32:00 UTC

[GitHub] [arrow] jorisvandenbossche opened a new pull request, #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

jorisvandenbossche opened a new pull request, #14246:
URL: https://github.com/apache/arrow/pull/14246

   Revert "ARROW-17614: [CI][Python] test test_write_dataset_max_rows_per_file is producing several nightly build failures (#14199)" (commit acd69f92ee92140f42b64714a348d0735a368931), as this causing new failures in `test_write_dataset_s3_put_only`


-- 
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] westonpace commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
westonpace commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1259697932

   Sorry, I'll get to the bottom of this.


-- 
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] westonpace commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
westonpace commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1260158152

   #14257 should fix the deadlock properly.


-- 
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] jorisvandenbossche commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
jorisvandenbossche commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1259527832

   https://issues.apache.org/jira/browse/ARROW-17853


-- 
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] jorisvandenbossche commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
jorisvandenbossche commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1259526062

   The Python builds on my fork have passed, eg https://github.com/jorisvandenbossche/arrow/actions/runs/3135613499/jobs/5091502221


-- 
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] ursabot commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
ursabot commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1260027118

   Benchmark runs are scheduled for baseline = 4ed79f7e3c2b9e83025042f396be5d60ce616020 and contender = a13032adec01fc156c66494f1ce763f6a89b83fc. a13032adec01fc156c66494f1ce763f6a89b83fc is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
   Conbench compare runs links:
   [Finished :arrow_down:0.0% :arrow_up:0.0%] [ec2-t3-xlarge-us-east-2](https://conbench.ursa.dev/compare/runs/5d9685a8f6a44d49ad752fab2cac258e...5248096bc0b8487a8ae8143ef903cef5/)
   [Failed :arrow_down:0.48% :arrow_up:0.0%] [test-mac-arm](https://conbench.ursa.dev/compare/runs/5f897a697415440abd26a79054731646...fb1782f3b6f84bd2bb2a5934cc3862de/)
   [Failed :arrow_down:0.27% :arrow_up:0.0%] [ursa-i9-9960x](https://conbench.ursa.dev/compare/runs/fa342df6bcf14a86bd7b2527783c7b15...1f8e04178c6a4780add3d917e94b6eb7/)
   [Finished :arrow_down:0.07% :arrow_up:0.0%] [ursa-thinkcentre-m75q](https://conbench.ursa.dev/compare/runs/8c903a0cbb944227b2437143784eaa26...8e677e0bf6c64d5dbae45e591ad2496a/)
   Buildkite builds:
   [Finished] [`a13032ad` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/1576)
   [Finished] [`a13032ad` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/1594)
   [Failed] [`a13032ad` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/1577)
   [Finished] [`a13032ad` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/1590)
   [Finished] [`4ed79f7e` ec2-t3-xlarge-us-east-2](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ec2-t3-xlarge-us-east-2/builds/1575)
   [Failed] [`4ed79f7e` test-mac-arm](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-test-mac-arm/builds/1593)
   [Failed] [`4ed79f7e` ursa-i9-9960x](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-i9-9960x/builds/1576)
   [Finished] [`4ed79f7e` ursa-thinkcentre-m75q](https://buildkite.com/apache-arrow/arrow-bci-benchmark-on-ursa-thinkcentre-m75q/builds/1589)
   Supported benchmarks:
   ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
   test-mac-arm: Supported benchmark langs: C++, Python, R
   ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
   ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java
   


-- 
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] jorisvandenbossche merged pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
jorisvandenbossche merged PR #14246:
URL: https://github.com/apache/arrow/pull/14246


-- 
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] jorisvandenbossche commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
jorisvandenbossche commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1259440173

   cc @westonpace unless you directly know what is going on to fix it properly, I would prefer reverting the original fix to at least have functional CI for Python


-- 
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] pitrou commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
pitrou commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1259483712

   @jorisvandenbossche Feel free to merge once CI looks green on your fork.
   


-- 
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] github-actions[bot] commented on pull request #14246: ARROW-17853: temporary revert fix for test_write_dataset_max_rows_per_file

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #14246:
URL: https://github.com/apache/arrow/pull/14246#issuecomment-1259599356

   :warning: Ticket **has not been started in JIRA**, please click 'Start Progress'.


-- 
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