You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@pig.apache.org by "Gianmarco De Francisci Morales (JIRA)" <ji...@apache.org> on 2012/10/28 19:59:11 UTC

[jira] [Commented] (PIG-3008) Fix whitespace in Pig code

    [ https://issues.apache.org/jira/browse/PIG-3008?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13485689#comment-13485689 ] 

Gianmarco De Francisci Morales commented on PIG-3008:
-----------------------------------------------------

I agree to fix whitespace, and the policy you suggest seems reasonable.
However, this conversation belongs to the more general topic of style.
Are we willing to define and enforce a code style policy for Pig?
Other projects are using checkstyle to do so, so that new patches comply.
I am fine with being stricter on style, but only for things that can be automated within an IDE (whitespace is one of those).
                
> Fix whitespace in Pig code
> --------------------------
>
>                 Key: PIG-3008
>                 URL: https://issues.apache.org/jira/browse/PIG-3008
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Jonathan Coveney
>             Fix For: 0.12
>
>
> This JIRA exists mainly to get a conversation started. We've talked about it before, and it's a tricky issue. That said, some of the Pig code is super, super gnarly. We need some sort of path that will let it eventually be fix-able.
> I posit: any file that hasn't been touched for over 6 months is eligible for a whitespace patch.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira