Skip to content

Conversation

@viirya
Copy link
Member

@viirya viirya commented Sep 5, 2015

It seems a typo that causes the test failed.

@viirya
Copy link
Member Author

viirya commented Sep 5, 2015

@yhuai @davies

@cloud-fan
Copy link
Contributor

LGTM, cc @liancheng

@rxin
Copy link
Contributor

rxin commented Sep 5, 2015

Ah thanks - I merged #8614 just now without seeing this one earlier.

Mind closing it now?

@viirya
Copy link
Member Author

viirya commented Sep 5, 2015

Ok.

@viirya viirya closed this Sep 5, 2015
@SparkQA
Copy link

SparkQA commented Sep 5, 2015

Test build #42041 has finished for PR 8612 at commit 8ca8a90.

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

@liancheng
Copy link
Contributor

@viirya Thanks anyway. Just would like to add that it's not a typo, but an unfortunate Git auto-merge conflict between #8584 and #8155.

@viirya viirya deleted the fix-sqlconfsuite branch December 27, 2023 18:32
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.

5 participants