-
Notifications
You must be signed in to change notification settings - Fork 560
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
Comments
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 |
Should you be looking for inspiration, this would be a really useful feature for us! |
@dmcg I'll try to make this available in 0.6 at the end of the month. I'll start testing next week. |
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 |
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. |
…om individual pom files to the main one.
Hey @mkjois, I just pushed a commit to the |
Merged upgraded changes into master. Closing this issue. |
@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 |
Could you provide an example? |
Nevermind. I got it working the problem which I got was in included |
hey @helloalleo, what problem were you running into? Was there a specific exception? |
I got this error when included
|
There are some breaking changes in the embedded server interface.
The text was updated successfully, but these errors were encountered: