You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@heron.apache.org by GitBox <gi...@apache.org> on 2022/07/07 22:21:42 UTC

[GitHub] [incubator-heron] nicknezis commented on pull request #3778: Joshfischer/3774/pom dependencies

nicknezis commented on PR #3778:
URL: https://github.com/apache/incubator-heron/pull/3778#issuecomment-1178297198

   But we were forcing API users to have to use the embedded classes (which could conflict). By removing the fat jar and providing the actual list of dependencies, this provides the ability to do proper dependency resolution, and provides the analytic developer more flexibility if they want to use different dependencies.
   
   BTW, this PR had some issues found in tested, I have an updated PR that I'm testing that fixes things a bit. https://github.com/apache/incubator-heron/pull/3844


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@heron.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org