Skip to content

[MLLIB] fix a unresolved reference variable 'nb' bug#2423

Closed
OdinLin wants to merge 1 commit into
apache:masterfrom
OdinLin:master
Closed

[MLLIB] fix a unresolved reference variable 'nb' bug#2423
OdinLin wants to merge 1 commit into
apache:masterfrom
OdinLin:master

Conversation

@OdinLin

@OdinLin OdinLin commented Sep 17, 2014

Copy link
Copy Markdown

variable nb is not reference in the raise log

@OdinLin OdinLin changed the title [SPARK-3550][MLLIB] fix a unresolved reference variable 'nb' bug [MLLIB] fix a unresolved reference variable 'nb' bug Sep 17, 2014
@SparkQA

SparkQA commented Sep 17, 2014

Copy link
Copy Markdown

Can one of the admins verify this patch?

@davies

davies commented Sep 17, 2014

Copy link
Copy Markdown
Contributor

@OdinLin good catch! But _common.py will be retired after PR #2378, maybe it's not needed anymore.

@OdinLin

OdinLin commented Sep 18, 2014

Copy link
Copy Markdown
Author

Got it!

@mengxr

mengxr commented Sep 19, 2014

Copy link
Copy Markdown
Contributor

@OdinLin Thanks for catching the bug! As @davies mentioned, #2378 will completely replace the current SerDe. Could you close this PR?

@asfgit asfgit closed this in d112a6c Sep 21, 2014
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