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

Bump major version to reflect backcompat break? #409

Closed
derenrich opened this issue Apr 20, 2017 · 8 comments
Closed

Bump major version to reflect backcompat break? #409

derenrich opened this issue Apr 20, 2017 · 8 comments

Comments

@derenrich
Copy link

Does this repo follow semantic versioning?

If so shouldn't we be on 3.0.0 now given remoting took a backcompat-break for version 2.3.0?

I just got bit by that backcompat break.

@markelliot
Copy link
Contributor

markelliot commented Apr 20, 2017 via email

@derenrich
Copy link
Author

derenrich commented Apr 20, 2017 via email

@markelliot
Copy link
Contributor

markelliot commented Apr 20, 2017 via email

@derenrich
Copy link
Author

derenrich commented Apr 20, 2017 via email

@markelliot
Copy link
Contributor

markelliot commented Apr 20, 2017 via email

@derenrich
Copy link
Author

I mean the change requires that user's of the library to install the Http2Agent whereas before it was installed for you.

  • The framework using remoting should be responsible for installing the
    Http2Agent (for example witchcraft or spark-module-lib), as to ensure
    that it is done early enough.

I hit this issue when upgrading from an older version of witchcraft. While I still think this should be bumped in retrospect at this point bumping it won't solve any problems so I'm closing this.

@ash211
Copy link
Contributor

ash211 commented Apr 20, 2017

I had to make two changes to accommodate this:

  • add dependency on compile 'com.palantir.remoting2:http2-agent'
  • put this as a static block on the class with my main method:
static {
    Http2Agent.install();
}

@markelliot
Copy link
Contributor

markelliot commented Apr 20, 2017 via email

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

No branches or pull requests

3 participants