Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port HappyPandas to java #49

Merged

Conversation

mahmoudhanafy
Copy link
Contributor

@mahmoudhanafy mahmoudhanafy commented May 18, 2016

#47

public DataFrame squishPandaFromPace(DataFrame pandaPlace) {
Buffer<Column> inputCols = JavaConversions.asScalaBuffer(Arrays.asList(pandaPlace.col("pandas")));

TypeTags.TypeTag tag = null; // TODO don't know how to create Type Tag in java ??
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@holdenk need help here

@mahmoudhanafy
Copy link
Contributor Author

I think squishPandaFromPace example hard to be done in Java.

@mahmoudhanafy mahmoudhanafy force-pushed the port-HappyPandas-to-Java branch 4 times, most recently from da8e64d to 906524b Compare May 19, 2016 05:59
@mahmoudhanafy mahmoudhanafy force-pushed the port-HappyPandas-to-Java branch 2 times, most recently from 578fc3a to df0139d Compare May 19, 2016 22:19
@codecov-io
Copy link

codecov-io commented May 19, 2016

Current coverage is 40.46%

Merging #49 into master will decrease coverage by 0.22%

@@             master        #49   diff @@
==========================================
  Files            23         23          
  Lines           430        430          
  Methods         350        350          
  Messages          0          0          
  Branches         12         12          
==========================================
- Hits            175        174     -1   
- Misses          255        256     +1   
  Partials          0          0          

Powered by Codecov. Last updated by 09b3d85...072c496

@mahmoudhanafy mahmoudhanafy force-pushed the port-HappyPandas-to-Java branch from df0139d to e792bb3 Compare May 20, 2016 14:53
@mahmoudhanafy mahmoudhanafy force-pushed the port-HappyPandas-to-Java branch from b96afd5 to b887283 Compare May 21, 2016 06:29
@mahmoudhanafy mahmoudhanafy force-pushed the port-HappyPandas-to-Java branch from 072c496 to d7de259 Compare May 21, 2016 07:31
@holdenk
Copy link
Member

holdenk commented May 23, 2016

This looks great! We should add an issue to track squishPandaFromPace to Java.

@holdenk holdenk merged commit 8a395d4 into high-performance-spark:master May 23, 2016
@mahmoudhanafy mahmoudhanafy deleted the port-HappyPandas-to-Java branch May 23, 2016 06:02
rachelwarren pushed a commit that referenced this pull request Jan 3, 2019
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.

3 participants