hawq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ming Li <...@pivotal.io>
Subject Re: enforce -Werror (if gcc) in hawq?
Date Mon, 05 Sep 2016 13:51:59 GMT
Good suggestion.

However, IMHO, we may need to firstly enable coverity scan check or clang
analysis scan. Also we should make the output of these check on a public
server so that all contributor can access them.

On Mon, Sep 5, 2016 at 6:05 PM, Paul Guo <paulguo@gmail.com> wrote:

> -Werror
> Make all warnings into errors.
> I've seen many cases (not just hawq) before that ignoring gcc warning leads
> to bugs. I'm wondering we should add the option for the gcc case. Given
> there may be a lot of warnings when building the common postgres code in
> hawq, we could at least enforce it in our own code at first
> (src/backend/cdb, src/backend/resourcemanager, src/test/feature, other
> directories?)? Any suggestion?
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message