Skip to content

Null handling in some transformers.#1

Open
igieon wants to merge 7 commits intomasterfrom
MinorImprovement
Open

Null handling in some transformers.#1
igieon wants to merge 7 commits intomasterfrom
MinorImprovement

Conversation

@igieon
Copy link
Collaborator

@igieon igieon commented Aug 23, 2018

Kafka source create df if new data arrive

Signed-off-by: David Ignjic ignjic@gmail.com

Kafka source create df if new data arrive

Signed-off-by: David Ignjic <ignjic@gmail.com>
@igieon igieon requested a review from redsk August 23, 2018 11:19

val linearCombination = udf {
(linearWeights:Row, values: Seq[Float]) => {
(values, linearWeights.getSeq[Float](1)).zipped.map(_ * _).sum / linearWeights.getFloat(0)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So what is the problem here? that linearWeights is shorter than values? In that case I think it should be addressed before calling the udf as it might drive you to believe this is behaving correctly when it's not.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Problem is that i want ignore null columns and also remove weights of it

igieon and others added 6 commits August 23, 2018 13:39
Signed-off-by: David Ignjic <ignjic@gmail.com>
Signed-off-by: David Ignjic <ignjic@gmail.com>
Signed-off-by: David Ignjic <ignjic@gmail.com>
Signed-off-by: David Ignjic <ignjic@gmail.com>
@redsk redsk force-pushed the master branch 3 times, most recently from 965de82 to 885c872 Compare September 18, 2018 12:04
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.

2 participants