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

WIP: Feat/pintracker revamp optracker sync map #399

Closed
wants to merge 28 commits into from

Conversation

lanzafame
Copy link
Contributor

Just testing the Windows jenkins build pipeline with sync.Map.

Don't review.

@ghost ghost assigned lanzafame May 1, 2018
@ghost ghost added the status/in-progress In progress label May 1, 2018
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
The TrackerStatuses were starting to be used to convey the inflight
status of an 'operation', instead of just the status of the Pin.
I have separated out any thing related to 'operations' and
an operation's 'phases'.

License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
to the ipfscluster.IPFSConnector interface and then
to the implementation of that interface in ipfsconn/ipfshttp.
This allows calls from MapPinTracker to cancel requests made
to the local IPFS node.

License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
come off the pin/unpin channels.

Also fix a race condition in the operationTracker.

License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
This reverts commit 41f6d9a.

License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT
Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
@lanzafame lanzafame force-pushed the feat/pintracker-revamp-optracker-sync-map branch from 31c761b to 3a08ca3 Compare May 2, 2018 02:30
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 67.887% when pulling 3a08ca3 on feat/pintracker-revamp-optracker-sync-map into a0a0898 on master.

@lanzafame
Copy link
Contributor Author

Closing as sync.Map was not required. See #383.

@lanzafame lanzafame closed this May 7, 2018
@ghost ghost removed the status/in-progress In progress label May 7, 2018
@lanzafame lanzafame deleted the feat/pintracker-revamp-optracker-sync-map branch May 7, 2018 03:55
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

Successfully merging this pull request may close these issues.

None yet

2 participants