Skip to content

Conversation

@icexelloss
Copy link
Contributor

@icexelloss icexelloss commented Oct 20, 2017

I open this PR so we can have a place to discuss the design.

We don't necessary need to merge a md file for the doc - this be can embeded python documentation.

@icexelloss
Copy link
Contributor Author

cc @cloud-fan @ueshin @HyukjinKwon @gatorsmile @viirya

To continue the discussion on #19505

@SparkQA
Copy link

SparkQA commented Oct 20, 2017

Test build #82935 has finished for PR 19544 at commit 3005312.

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

@gatorsmile
Copy link
Member

@jiangxb1987 will reorg the existing Spark SQL doc. We can think about how to put this into the new version of Spark SQL doc.

@srowen
Copy link
Member

srowen commented Oct 24, 2017

I don't think we would merge this to Spark docs right? I don't think this is an efficient way to discuss a design document if that's the intent

@gatorsmile
Copy link
Member

This is not ready yet. Eventually, we need a document for vectorized Python UDF

@icexelloss
Copy link
Contributor Author

Sorry for the late reply.

@srowen At this point I would like to have feedback on the design doc so we can move forward with implementation or a formal Spark docs if people prefer. What do you think is a best way to discuss design document?

@icexelloss
Copy link
Contributor Author

Do people prefer google doc? I can do that too.

@cloud-fan
Copy link
Contributor

I feel google doc is easier to review and discuss

@icexelloss
Copy link
Contributor Author

icexelloss commented Oct 25, 2017 via email

@icexelloss
Copy link
Contributor Author

Thanks all for the advice. I've put this in a google doc.

https://docs.google.com/document/d/1KlLaa-xJ3oz28xlEJqXyCAHU3dwFYkFs_ixcUXrJNTc/edit?usp=sharing

@icexelloss icexelloss closed this Nov 16, 2017
@icexelloss
Copy link
Contributor Author

Close this in favor of the google doc.

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