Skip to content

Conversation

@yhuai
Copy link
Contributor

@yhuai yhuai commented Jun 1, 2016

What changes were proposed in this pull request?

Since we are pretty late in the 2.0 release cycle, it is not clear if this upgrade can be tested thoroughly and if we can resolve the regression issue that we observed before. This PR reverts #13280 from branch 2.0.

How was this patch tested?

Existing tests

This reverts commit 776d183.

@yhuai
Copy link
Contributor Author

yhuai commented Jun 1, 2016

@rdblue @liancheng Can you review this PR? This is for branch 2.0.

@rdblue
Copy link
Contributor

rdblue commented Jun 1, 2016

+1, looks fine to me assuming tests pass.

@liancheng
Copy link
Contributor

LGTM

@SparkQA
Copy link

SparkQA commented Jun 1, 2016

Test build #59764 has finished for PR 13450 at commit 0c08f31.

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

@SparkQA
Copy link

SparkQA commented Jun 1, 2016

Test build #59766 has finished for PR 13450 at commit 1d52cfb.

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

@liancheng
Copy link
Contributor

Merging to branch-2.0.

asfgit pushed a commit that referenced this pull request Jun 1, 2016
… 1.8.1."

## What changes were proposed in this pull request?
Since we are pretty late in the 2.0 release cycle, it is not clear if this upgrade can be tested thoroughly and if we can resolve the regression issue that we observed before. This PR reverts #13280 from branch 2.0.

## How was this patch tested?
Existing tests

This reverts commit 776d183.

Author: Yin Huai <[email protected]>

Closes #13450 from yhuai/revertParquet1.8.1-branch-2.0.
@yhuai yhuai closed this Jun 7, 2016
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.

4 participants