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

Pipeline graph timing & structural analysis APIs #6

Closed
wants to merge 78 commits into from

Conversation

@svanoort
Copy link
Member

commented Jun 10, 2016

Not ready for review, please respect the WIP label.

Builds on top of #2 with a set of data structures to split into chunks (runs of nodes with a defined start & end -- they may be the same node though).

Couples with the https://github.com/jenkinsci/pipeline-graph-analysis-plugin which will then provide status/timing on the chunks.

Consumers:

  • Implement SimpleBlockVisitor to consume SAX-like notifications of blocks & parallels info, and use the ForkScanner.visitSimpleChunks method to invoke it.
  • Maybe use a standardized callback-based API that builds on the simple visitors to couple with a storage system for chunks (or maybe even return a full representation of the flow).
svanoort added 30 commits Apr 14, 2016
Refactor flow graph incremental analysis, add stub of a test until I …
…can figure out how to get it to use something like semaphore step
FlowScanner: switch to simpler iterator use internally, add filterato…
…r, rewrite internals for iterator, add block test
Massive refactor: give graph analysis its own package, and split the …
…different flow scanners into individual classes
Clean up from initial review: add and fix javadocs, refactor some fie…
…ld names, make visitAll method take a blackList and add helper for it

@svanoort svanoort closed this Aug 12, 2016

@svanoort

This comment has been minimized.

Copy link
Member Author

commented Aug 12, 2016

Replaced by #10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.