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

Add memory benchmark for validatelargetx #2659

Merged
merged 1 commit into from Oct 12, 2017

Conversation

Projects
None yet
3 participants
@str4d
Contributor

str4d commented Oct 9, 2017

Closes #2377.

@daira

daira approved these changes Oct 10, 2017

utACK

@str4d

This comment has been minimized.

Show comment
Hide comment
@str4d

str4d Oct 12, 2017

Contributor

Gonna give this a trivial self-ACK.

Contributor

str4d commented Oct 12, 2017

Gonna give this a trivial self-ACK.

@str4d

This comment has been minimized.

Show comment
Hide comment
@str4d

str4d Oct 12, 2017

Contributor

@zkbot r+

Contributor

str4d commented Oct 12, 2017

@zkbot r+

@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Oct 12, 2017

Contributor

📌 Commit 94790c5 has been approved by str4d

Contributor

zkbot commented Oct 12, 2017

📌 Commit 94790c5 has been approved by str4d

@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Oct 12, 2017

Contributor

⌛️ Testing commit 94790c5 with merge 9f9d552...

Contributor

zkbot commented Oct 12, 2017

⌛️ Testing commit 94790c5 with merge 9f9d552...

zkbot added a commit that referenced this pull request Oct 12, 2017

Auto merge of #2659 - str4d:2377-memory-validatelargetx, r=str4d
Add memory benchmark for validatelargetx

Closes #2377.
@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Oct 12, 2017

Contributor

💔 Test failed - pr-merge

Contributor

zkbot commented Oct 12, 2017

💔 Test failed - pr-merge

@str4d

This comment has been minimized.

Show comment
Hide comment
@str4d

str4d Oct 12, 2017

Contributor

Transient failure.

@zkbot retry

Contributor

str4d commented Oct 12, 2017

Transient failure.

@zkbot retry

@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Oct 12, 2017

Contributor

⌛️ Testing commit 94790c5 with merge 4ad2f17...

Contributor

zkbot commented Oct 12, 2017

⌛️ Testing commit 94790c5 with merge 4ad2f17...

zkbot added a commit that referenced this pull request Oct 12, 2017

Auto merge of #2659 - str4d:2377-memory-validatelargetx, r=str4d
Add memory benchmark for validatelargetx

Closes #2377.
@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Oct 12, 2017

Contributor

☀️ Test successful - pr-merge
Approved by: str4d
Pushing 4ad2f17 to master...

Contributor

zkbot commented Oct 12, 2017

☀️ Test successful - pr-merge
Approved by: str4d
Pushing 4ad2f17 to master...

@zkbot zkbot merged commit 94790c5 into zcash:master Oct 12, 2017

1 check passed

homu Test successful
Details

@str4d str4d deleted the str4d:2377-memory-validatelargetx branch Oct 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment