[SPARK-11551][DOC][Example]Replace example code in ml-features.md using include_example

[somideshmukh]

Made new patch contaning only markdown examples moved to exmaple/folder.
Ony three java code were not shfted since they were contaning compliation error ,these classes are
1)StandardScale 2)NormalizerExample 3)VectorIndexer

[AmplabJenkins]

Can one of the admins verify this patch?

[yinxusen]

Hi @somideshmukh, since it's lots of files in this pull request, I send a pull request to your repo and you can see it here: somideshmukh#1

All of my modifications are there, you can check the code and merge it if it looks good to you. Then we can check the correctness of those newly added files.

[yinxusen]

You can see the files changed in that pull request. I make changes in your original code. The main issues there are:

• The mismatch between file name and app name
• The indention in code is not very correct
• You forget to add // $example on$ and // $example off$ pairs in the code
• The reorganize of imports

I also add Python files that you missed and 3 Java codes plus with 1 Scala code.

[yinxusen]

Besides, there is no need to create a new pull request every time you changing your code. You can modify your previous code and push the changes in current branch.

[yinxusen]

Pls remove this line

[yinxusen]

@somideshmukh This is really a bunch of code. I check some errors as described above for my carelessness and sorry for that. And after fixing those errors, I can get it run in my computer. I'll also check the example codes one by one to make sure every code example is runnable without errors.

[yinxusen]

I think the conflict is due to some changes in ml-features.md, but we can leave it for now. After we fix all errors we can fix the conflict.

[yinxusen]

add a white space here " categorical features:", otherwise the output is like this Chose 351categorical features: ...

[somideshmukh]

I have made the changes you have mentioned in the given Java and Scala files and push the changes in Branch "SomilBranch1.33". I haven't modified Python files since I haven't work on that files and I donot have python knowledge.

[somideshmukh]

Pls check it and let me know whether you have got the changes.

[srowen]

@somideshmukh the changes are here as you can see, but your branch still needs a rebase.

[yinxusen]

@somideshmukh I'll help you change the python issues, then I will send another PR to you soon.

[yinxusen]

@somideshmukh I have already fix all issues and merged with master. You can accept my pull request here somideshmukh#2. Even though it looks very large, it can be small when you merging it. I believe we can merge the PR after that.

[yinxusen]

@mengxr LGTM

[yinxusen]

@mengxr Shall we merge this PR? I find there are also some PRs modify ml-features.md, so I think it's better to merge it before new conflicts emerging.

[mengxr]

minor: remove extra } at the end. You can submit a follow-up PR to fix it.

[yinxusen]

@mengxr Yes, I'll submit a follow-up pr to refine it.

[mengxr]

Merged into master and branch-1.6. Thanks!

[JoshRosen]

Looks like this broke the Python style tests:

PEP8 checks failed.
./examples/src/main/python/ml/binarizer_example.py:41:4: E114 indentation is not a multiple of four (comment)
./examples/src/main/python/ml/onehot_encoder_example.py:39:1: W293 blank line contains whitespace
./examples/src/main/python/ml/pca_example.py:33:9: E128 continuation line under-indented for visual indent
./examples/src/main/python/ml/pca_example.py:35:41: E231 missing whitespace after ','
./examples/src/main/python/ml/polynomial_expansion_example.py:34:9: E128 continuation line under-indented for visual indent
./examples/src/main/python/ml/polynomial_expansion_example.py:35:9: E128 continuation line under-indented for visual indent

I guess that Jenkins never tested this PR?

[liancheng]

Reverting this one to bring back Spark builds.

[yinxusen]

Sorry for that, I will help fixing them.

[yinxusen]

@somideshmukh I give you another PR to fix the Python style issue: somideshmukh#3. After merging it, we can call jenkins testing once here.

[liancheng]

Hit some network issue, just reverted this PR from master and branch-1.6 in #10200.

[yinxusen]

@somideshmukh Do you still have time work on this? Pleas let me know ASAP thanks.

[liancheng]

@yinxusen Maybe you can fork this PR branch and get it merged. You may add a note in your PR description to remind committers to attribute your PR to @somideshmukh. We can specify primary author while merging a PR using our merge script.

[yinxusen]

Sure, I will do it

[somideshmukh]

Please let me know what needs to be done,I have not worked on python code.

[yinxusen]

@somideshmukh Never mind, I add a new one on behalf of you: #10219@liancheng if it looks good to you, please help me merge it after the Jenkins test. Thanks!

[liancheng]

@yinxusen Well, I'm probably too ignorant to review #10219 :) I'd ping @mengxr to sign it off.