You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "liyunzhang_intel (JIRA)" <ji...@apache.org> on 2015/07/03 03:46:04 UTC

[jira] [Updated] (PIG-4619) Cleanup: change the indent size of some files of pig on spark project from 2 to 4 space

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

liyunzhang_intel updated PIG-4619:
----------------------------------
    Attachment: PIG-4619.patch

[~mohitsabharwal],[~kexianda],[~praveenr019],[~xuefuz]:
changes in PIG-4619.patch:
1.change the indent size from 2 space to 4 space in following files:
org.apache.pig.backend.hadoop.executionengine.spark.converter.CollectedGroupConverter
org.apache.pig.backend.hadoop.executionengine.spark.JobMetricsListener
org.apache.pig.backend.hadoop.executionengine.spark.SparkLocalExecType

2. Previously, SparkLauncher.java uses tab to replace 4 space.  The code style of this file is not same with other files. This is changed in the patch.



> Cleanup: change the indent size of some files of pig on spark project from 2 to 4 space
> ---------------------------------------------------------------------------------------
>
>                 Key: PIG-4619
>                 URL: https://issues.apache.org/jira/browse/PIG-4619
>             Project: Pig
>          Issue Type: Sub-task
>          Components: spark
>            Reporter: liyunzhang_intel
>            Assignee: liyunzhang_intel
>             Fix For: spark-branch
>
>         Attachments: PIG-4619.patch
>
>
> following files under pig on spark project use 2 space indent:
> org.apache.pig.backend.hadoop.executionengine.spark.converter.CollectedGroupConverter
> org.apache.pig.backend.hadoop.executionengine.spark.JobMetricsListener
> org.apache.pig.backend.hadoop.executionengine.spark.SparkLocalExecType
> Now all the files under this project should use 4 space indent.
> Besides SparkLauncher.java use tab to replace space.  We don't use tab to replace space in all the files in this project so need change this file.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)