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)