Skip to content

Conversation

@akharit
Copy link
Member

@akharit akharit commented Oct 12, 2018


This checklist is used to make sure that common guidelines for a pull request are followed.

Description of the change

This pr contains recursive acl functions. It uses both multi-processing and multi-threading due to limitations in Python threading model because of Global Interpreter Lock.

General Guidelines

  • The PR has modified HISTORY.rst with an appropriate description of the change and a version increment.
    Will be added when merge to master from dev.
  • The PR has supporting test coverage that confirm the expected behavior and protects against regressions, including necessary recordings.
  • Links to associated bugs, if any, are in the description.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.4%) to 87.15% when pulling c06fe6a on akharit-multProcRecAcl into c80bc96 on dev.

@coveralls
Copy link

coveralls commented Oct 12, 2018

Coverage Status

Coverage increased (+0.08%) to 86.571% when pulling cda72f8 on akharit-multProcRecAcl into 96c064b on dev.

@akharit
Copy link
Member Author

akharit commented Oct 15, 2018

Encountering some timing issues which don't allow some cleanup function to block indefinitely.

@rahuldutta90 rahuldutta90 changed the base branch from dev to master November 6, 2018 01:28
@rahuldutta90 rahuldutta90 changed the base branch from master to dev November 6, 2018 01:28
@rahuldutta90 rahuldutta90 changed the base branch from dev to master November 6, 2018 01:29
@akharit akharit changed the base branch from master to dev November 6, 2018 01:36
@akharit akharit merged commit 2628e49 into dev Nov 9, 2018
@akharit akharit deleted the akharit-multProcRecAcl branch November 15, 2018 02:11
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.

4 participants