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

Support for Spark 2.6.0 #42

Closed
sapessi opened this issue Jul 3, 2017 · 13 comments
Closed

Support for Spark 2.6.0 #42

sapessi opened this issue Jul 3, 2017 · 13 comments
Assignees
Milestone

Comments

@sapessi
Copy link
Collaborator

sapessi commented Jul 3, 2017

There are some breaking changes in the embedded server interface.

@mkjois
Copy link

mkjois commented Jul 3, 2017

For reference, their changelog is here: http://sparkjava.com/news#spark-26-released

And the full commit history between 2.5.5 and 2.6.0: perwendel/spark@2.5.5...2.6.0

@dmcg
Copy link

dmcg commented Jul 13, 2017

Should you be looking for inspiration, this would be a really useful feature for us!

@sapessi sapessi added this to the Release 0.6 milestone Jul 13, 2017
@sapessi
Copy link
Collaborator Author

sapessi commented Jul 13, 2017

@dmcg I'll try to make this available in 0.6 at the end of the month. I'll start testing next week.

@sapessi
Copy link
Collaborator Author

sapessi commented Aug 8, 2017

@dmcg @mkjois I've looked through the changes, they are minor but still breaking. I have two options:

  1. Upgrade to spark 2.6.0 for all
  2. Add a new module specifically for spark2.6.0

What would your preference be?

@sapessi sapessi self-assigned this Aug 8, 2017
@mkjois
Copy link

mkjois commented Aug 8, 2017

My preference would be to upgrade to spark 2.6.0 for all, either in version 0.6 or 0.7.

I can understand option 2 but if you do go with it I think this module should continue to mature with Spark, while you can introduce a new module for staying with 2.5.3

@sapessi
Copy link
Collaborator Author

sapessi commented Aug 8, 2017

Thanks @mkjois - I will publish 0.6 as is since I have a bug fix for the servlet specs that should be included in all releases. From 0.7-SNAPSHOT I will upgrade to spark 2.6.0 for all.

@sapessi sapessi modified the milestones: Release 0.7, Release 0.6 Aug 8, 2017
sapessi added a commit that referenced this issue Aug 15, 2017
@sapessi
Copy link
Collaborator Author

sapessi commented Aug 15, 2017

Hey @mkjois, I just pushed a commit to the spark-upgrade branch that supports 2.6.0. If you get a chance, can you give it a try?

@sapessi
Copy link
Collaborator Author

sapessi commented Aug 16, 2017

Merged upgraded changes into master. Closing this issue.

@sapessi sapessi closed this as completed Aug 16, 2017
@mkjois
Copy link

mkjois commented Aug 16, 2017

@sapessi Sorry didn't get to test, I'm pretty swamped at work this week.

Thanks for the upgrade though! Might want to update the readme with a note about these versions as well

@lynx-r
Copy link

lynx-r commented Sep 27, 2017

Could you provide an example?

@lynx-r
Copy link

lynx-r commented Sep 27, 2017

Nevermind. I got it working the problem which I got was in included compile 'com.sparkjava:spark-debug-tools:0.5'

@sapessi
Copy link
Collaborator Author

sapessi commented Sep 27, 2017

hey @helloalleo, what problem were you running into? Was there a specific exception?

@lynx-r
Copy link

lynx-r commented Sep 27, 2017

I got this error when included compile 'com.sparkjava:spark-debug-tools:0.5'

java.lang.ClassCastException: spark.SparkInstance cannot be cast to spark.Service
at com.amazonaws.serverless.proxy.spark.SparkLambdaContainerHandler.<init>(SparkLambdaContainerHandler.java:120)
at com.amazonaws.serverless.proxy.spark.SparkLambdaContainerHandler.getAwsProxyHandler(SparkLambdaContainerHandler.java:92)
at com.workingbit.article.LambdaHandler.handleRequest(LambdaHandler.java:19)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants