-
Notifications
You must be signed in to change notification settings - Fork 153
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
Maven via JitPack: amazon-kinesis-aggregator-2.0.2 has v1 code, amazon-kinesis-deaggregator-2.0.2 unavailable #120
Comments
We're also manually downloading the release right now - which won't do for long as it's spreading like wildfire. It would be awesome if we could simply have the aggregator- and deaggregator-libraries in maven! |
@IanMeyers any chance we could get the v1 and v2 releases published out to a maven repository again? The most recent and only build currently is https://mvnrepository.com/artifact/com.amazonaws/amazon-kinesis-deaggregator/1.0.3 |
+1 |
Working on it - understand the ask and we'll hopefully have an update this week. |
Do we have any idea why JitPack is including the wrong version of the classes? Some parts of the 2.x codebase have a v1.x dependency but only for testing, and are not included in the jar as far as I can see. |
So this isn't exactly what you asked for, but I think it will maybe help in the short run. I've drafted 4 new releases: Version 2.x Version 1.x I hope that this helps everyone get the correct version through JitPack while we sort out push to Maven Central. |
Possibly because the v1 and v2 modules / artifacts have the same group / artifact ids and only differ in the version number? I'm guessing they did not expect a project to publish out multiple concurrent versions of unqiue groupId:artifactId combinations in the same build :) |
Yep that's why - didn't edit my comments after doing the research. That's why I've pushed new release labels that should address this. |
I saw you created two tags (2.0.3-agg and 2.0.3-deagg), but they point to the same commit hash. It appears both builds fail the same way on Jitpack, here are the build logs: The error message is: Haven't dug in, but seems like perhaps your multi-module build isn't able to find the siblings? Also, if you navigate to e.g. https://jitpack.io/#awslabs/kinesis-aggregation/2.0.3-deagg and scroll down to "Step 2", you'll see that the artifact selection drop-down only shows the aggregator (makes sense, since the build failed after building the aggregator). You'll also notice that a version selection (1.1.3/2.0.3) isn't possible. I haven't tried it, but it will likely download unpredictable versions again. Here's a screenshot of the drop-down as it is right now for reference: |
Hrm yes - was expecting jitpack to just use the Assets from the release, as opposed to something else. Will do more research. |
Any update on this? |
any update on publishing to central?? |
jitpack doesn't seem to work with the 1.x version, so this is really quite painful. |
Unfortunately right now we are unable to publish to Maven because of the package structure: the project is community maintained, but the signing certs for Maven are part of the build system maintained by the Kinesis service team. The only alternative is to change the top-level package structure, which would be a huge impact on our user-base. I appreciate the pain that this creates, and we will continue to try and achieve this in future. |
@IanMeyers would it be possible for you to publish it as a package in Github? It's not as good as Maven Central but it would solve the problem for a lot of people. |
We are looking into this and will come back to you. |
Any plan to update on publishing to central?? |
+1 |
I would say you might want to repackage anyway, since the |
Gradle config to ensure only this library is resolved from jitpack, not your entire dependency set:
|
I'm using the following in a Maven project by copy-pasting the code from:
i.e.
I would much rather just refer to these packages via Maven dependencies. Since they don't appear to be in a Maven repo, I've attempted to use JitPack to pull in the GitHub releases:
https://jitpack.io/#awslabs/kinesis-aggregation/2.0.2 instructs adding the following to my POM:
amazon-kinesis-aggregator (v2)
This works to pull in
amazon-kinesis-aggregator-2.0.2.jar
, but it pulls in the classes (AggRecord
andRecordAggregator
) from the v1 submodule https://github.com/awslabs/kinesis-aggregation/tree/master/java/KinesisDeaggregator, which has the v1 variant methods, e.g.toPutRecordRequest
(v1) instead oftoPutRecordsRequest
(v2):amazon-kinesis-deaggregator (v2)
Attempting to pull in the
KinesisDeaggregatorV2
classes via JitPack fails to even find the JAR:TL;DR: my code works with the copy-pasted classes from
KinesisAggregatorV2
andKinesisDeaggregatorV2
, but I would like to load them as Maven dependencies instead. Is there a way I can do that?The text was updated successfully, but these errors were encountered: