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 2020/08/18 11:07:24 UTC
[GitHub] [arrow] kszucs opened a new pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
kszucs opened a new pull request #7986:
URL: https://github.com/apache/arrow/pull/7986
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kiszk commented on pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
kiszk commented on pull request #7986:
URL: https://github.com/apache/arrow/pull/7986#issuecomment-675593712
sounds good to me
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kou commented on pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
kou commented on pull request #7986:
URL: https://github.com/apache/arrow/pull/7986#issuecomment-675866692
Thanks.
We can work on the issue as a separated task. (We will disable the test with valgrind or disable AVX512 for valgrind build.)
I'll merge 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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #7986:
URL: https://github.com/apache/arrow/pull/7986#issuecomment-675416885
Revision: 8a188f07e5958bc50f54d7977046c3230245b517
Submitted crossbow builds: [ursa-labs/crossbow @ actions-491](https://github.com/ursa-labs/crossbow/branches/all?query=actions-491)
|Task|Status|
|----|------|
|test-conda-cpp-valgrind|[](https://github.com/ursa-labs/crossbow/actions?query=branch:actions-491-github-test-conda-cpp-valgrind)|
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #7986:
URL: https://github.com/apache/arrow/pull/7986#issuecomment-675418952
https://issues.apache.org/jira/browse/ARROW-9781
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #7986:
URL: https://github.com/apache/arrow/pull/7986#issuecomment-676051841
I've [bumbed](https://github.com/conda-forge/valgrind-feedstock/pull/11) the valgrind version in the conda forge feedstock, perhaps that will help.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] jianxind commented on pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
jianxind commented on pull request #7986:
URL: https://github.com/apache/arrow/pull/7986#issuecomment-675806083
The failed compute case is running with SimdLevel::AVX512(4), probably Valgrind doesn't support AVX512?
```
==5971== Your program just tried to execute an instruction that Valgrind
==5971== did not recognise. There are two possible reasons for this.
==5971== 1. Your program has a bug and erroneously jumped to a non-code
==5971== location. If you are running Memcheck and you just saw a
==5971== warning about a bad jump, it's probably your program's fault.
==5971== 2. The instruction is legitimate but Valgrind doesn't handle it,
==5971== i.e. it's Valgrind's fault. If you think this is the case or
==5971== you are not sure, please let us know and we'll try to fix it.
==5971== Either way, Valgrind will now raise a SIGILL signal which will
==5971== probably kill your program.
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kou closed pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
kou closed pull request #7986:
URL: https://github.com/apache/arrow/pull/7986
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7986: ARROW-9781: [C++] Fix valgrind uninitialized value warnings
Posted by GitBox <gi...@apache.org>.
kszucs commented on pull request #7986:
URL: https://github.com/apache/arrow/pull/7986#issuecomment-675414540
@github-actions crossbow submit test-conda-cpp-valgrind
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org