[jira] [Created] (ARROW-1185) [C++] Clean up arrow::Status implementation, add warn_unused_result attribute for clang

2017-07-05 Thread Wes McKinney (JIRA) 
Wes McKinney created ARROW-1185:
---

 Summary: [C++] Clean up arrow::Status implementation, add 
warn_unused_result attribute for clang
 Key: ARROW-1185
 URL: https://issues.apache.org/jira/browse/ARROW-1185
 Project: Apache Arrow
  Issue Type: Improvement
  Components: C++
Reporter: Wes McKinney
Assignee: Wes McKinney
 Fix For: 0.5.0






--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Created] (ARROW-1186) [C++] Enable option to build arrow with minimal dependencies needed to build Parquet library

2017-07-05 Thread Deepak Majeti (JIRA) 
Deepak Majeti created ARROW-1186:


 Summary: [C++] Enable option to build arrow with minimal 
dependencies needed to build Parquet library
 Key: ARROW-1186
 URL: https://issues.apache.org/jira/browse/ARROW-1186
 Project: Apache Arrow
  Issue Type: Bug
  Components: C++
Reporter: Deepak Majeti


As discussed in https://github.com/apache/parquet-cpp/pull/366, the scope of 
this JIRA is to enable a build option in Arrow to enable minimal transitive 
dependencies required for Parquet.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Resolved] (ARROW-1185) [C++] Clean up arrow::Status implementation, add warn_unused_result attribute for clang

2017-07-05 Thread Wes McKinney (JIRA) 

 [ 
https://issues.apache.org/jira/browse/ARROW-1185?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wes McKinney resolved ARROW-1185.
-
Resolution: Fixed

Issue resolved by pull request 814
[https://github.com/apache/arrow/pull/814]

> [C++] Clean up arrow::Status implementation, add warn_unused_result attribute 
> for clang
> ---
>
> Key: ARROW-1185
> URL: https://issues.apache.org/jira/browse/ARROW-1185
> Project: Apache Arrow
>  Issue Type: Improvement
>  Components: C++
>Reporter: Wes McKinney
>Assignee: Wes McKinney
> Fix For: 0.5.0
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Resolved] (ARROW-111) [C++] Add static analyzer to tool chain to verify checking of Status returns

2017-07-05 Thread Wes McKinney (JIRA) 

 [ 
https://issues.apache.org/jira/browse/ARROW-111?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wes McKinney resolved ARROW-111.

   Resolution: Fixed
 Assignee: Wes McKinney
Fix Version/s: 0.5.0

This was addressed by ARROW-1185 
https://github.com/apache/arrow/commit/00a7d55cc06ab54af860be38fea701169b8f8aa3 
-- Clang provides the {{warn_unused_result}} attribute so that unchecked 
Statuses will result in a compiler warning. This combined with {{-Werror}} will 
allow us to fail builds for unchecked Status

> [C++] Add static analyzer to tool chain to verify checking of Status returns
> 
>
> Key: ARROW-111
> URL: https://issues.apache.org/jira/browse/ARROW-111
> Project: Apache Arrow
>  Issue Type: Wish
>  Components: C++
>Reporter: Micah Kornfield
>Assignee: Wes McKinney
> Fix For: 0.5.0
>
>
> From Wes. Kudu wrote an LLVM static analyzer for this 
> https://github.com/apache/incubator-kudu/tree/master/build-support/tools/kudu-lint



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)