Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented May 28, 2015

So we can enable a whitespace enforcement rule in the style checker to save code review time.

@SparkQA
Copy link

SparkQA commented May 29, 2015

Test build #33691 has finished for PR 6478 at commit e01b0e0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@asfgit asfgit closed this in ee6a0e1 May 29, 2015
asfgit pushed a commit that referenced this pull request May 29, 2015
So we can enable a whitespace enforcement rule in the style checker to save code review time.

Author: Reynold Xin <[email protected]>

Closes #6478 from rxin/whitespace-hive and squashes the following commits:

e01b0e0 [Reynold Xin] Fixed tests.
a3bba22 [Reynold Xin] [SPARK-7927] whitespace fixes for Hive and ThriftServer.

(cherry picked from commit ee6a0e1)
Signed-off-by: Reynold Xin <[email protected]>
asfgit pushed a commit that referenced this pull request May 29, 2015
This is the last batch of changes to complete SPARK-7929.

Previous related PRs:
#6480
#6478
#6477
#6476
#6475
#6474
#6473

Author: Reynold Xin <[email protected]>

Closes #6487 from rxin/whitespace-lint and squashes the following commits:

b33d43d [Reynold Xin] [SPARK-7929] Turn whitespace checker on for more token types.
asfgit pushed a commit that referenced this pull request May 29, 2015
This is the last batch of changes to complete SPARK-7929.

Previous related PRs:
#6480
#6478
#6477
#6476
#6475
#6474
#6473

Author: Reynold Xin <[email protected]>

Closes #6487 from rxin/whitespace-lint and squashes the following commits:

b33d43d [Reynold Xin] [SPARK-7929] Turn whitespace checker on for more token types.

(cherry picked from commit 97a60cf)
Signed-off-by: Reynold Xin <[email protected]>
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
So we can enable a whitespace enforcement rule in the style checker to save code review time.

Author: Reynold Xin <[email protected]>

Closes apache#6478 from rxin/whitespace-hive and squashes the following commits:

e01b0e0 [Reynold Xin] Fixed tests.
a3bba22 [Reynold Xin] [SPARK-7927] whitespace fixes for Hive and ThriftServer.
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
This is the last batch of changes to complete SPARK-7929.

Previous related PRs:
apache#6480
apache#6478
apache#6477
apache#6476
apache#6475
apache#6474
apache#6473

Author: Reynold Xin <[email protected]>

Closes apache#6487 from rxin/whitespace-lint and squashes the following commits:

b33d43d [Reynold Xin] [SPARK-7929] Turn whitespace checker on for more token types.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
So we can enable a whitespace enforcement rule in the style checker to save code review time.

Author: Reynold Xin <[email protected]>

Closes apache#6478 from rxin/whitespace-hive and squashes the following commits:

e01b0e0 [Reynold Xin] Fixed tests.
a3bba22 [Reynold Xin] [SPARK-7927] whitespace fixes for Hive and ThriftServer.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
This is the last batch of changes to complete SPARK-7929.

Previous related PRs:
apache#6480
apache#6478
apache#6477
apache#6476
apache#6475
apache#6474
apache#6473

Author: Reynold Xin <[email protected]>

Closes apache#6487 from rxin/whitespace-lint and squashes the following commits:

b33d43d [Reynold Xin] [SPARK-7929] Turn whitespace checker on for more token types.
ckadner pushed a commit to ckadner/bahir_from_spark_8301fad that referenced this pull request Jun 6, 2016
This is the last batch of changes to complete SPARK-7929.

Previous related PRs:
apache/spark#6480
apache/spark#6478
apache/spark#6477
apache/spark#6476
apache/spark#6475
apache/spark#6474
apache/spark#6473

Author: Reynold Xin <[email protected]>

Closes #6487 from rxin/whitespace-lint and squashes the following commits:

b33d43d [Reynold Xin] [SPARK-7929] Turn whitespace checker on for more token types.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants