Skip to content

Don't use jobs. Instead be smart with ArtifactLoadState#1

Closed
kitskub wants to merge 3 commits intomasterfrom
state
Closed

Don't use jobs. Instead be smart with ArtifactLoadState#1
kitskub wants to merge 3 commits intomasterfrom
state

Conversation

@kitskub
Copy link
Contributor

@kitskub kitskub commented Aug 12, 2014

No description provided.

Where "artifact" means something that contains code, eg. a jar, a maven
artifact, a folder with classes.

I'm starting with state tracking, later I'll add actual means of loading
the code from artifacts.
@kitskub
Copy link
Contributor Author

kitskub commented Aug 12, 2014

@Wolf480pl

Why do you want all jobs to be processed?

@Wolf480pl
Copy link
Contributor

I want them all to be processed because I don't like the idea that someone will call artifactManager.someMethod("myArtifact") and by accident/race this will have no effect and the future will never become "done" (assigned a value or an exception)

@Wolf480pl
Copy link
Contributor

Renaming master to artifact and creating new master 2 commits earlier completely screwed this PR, and there's no way to change "into master from state" to "into artifact from state". I'm closing it I guess.

@Wolf480pl Wolf480pl closed this Aug 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants