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

Develop #1643

Merged
merged 12 commits into from
Dec 5, 2018
Merged

Develop #1643

merged 12 commits into from
Dec 5, 2018

Conversation

faustbrian
Copy link
Contributor

No description provided.

Kristjan Kosic and others added 12 commits December 4, 2018 15:10
…1636)

Impact on queries:

SELECT block_id, serialized FROM transactions ORDER BY timestamp DESC OFFSET 0 LIMIT 25;
~1000ms -> ~100ms

SELECT
type,
MIN(fee) AS minFee,
MAX(fee) AS maxFee,
AVG(fee) AS avgFee,
MAX(timestamp) AS timestamp
FROM transactions
WHERE timestamp >= 53823000
GROUP BY type
ORDER BY timestamp DESC;
~350ms -> ~50ms
Add indexes on transactions's sender_public_key and recipient_id
columns.

Impact on queries:

SELECT count(*) AS count FROM transactions WHERE
sender_public_key = '... many matches ...' OR
recipient_id = '... many matches ...';
~230ms -> 230ms (no change)

SELECT count(*) AS count FROM transactions WHERE
sender_public_key = '... many matches ...' OR
recipient_id = '... few matches ...';
~230ms -> 90ms

SELECT count(*) AS count FROM transactions WHERE
sender_public_key = '... few matches ...' OR
recipient_id = '... many matches ...';
~230ms -> 70ms

SELECT count(*) AS count FROM transactions WHERE
sender_public_key = '... few matches ...' OR
recipient_id = '... few matches ...';
~230ms -> 50ms
* feat: supply calculator

* test: supply calculator

* refactor: use supply calculator vote report

* refactor: method name

* refactor: return number
@ghost ghost assigned faustbrian Dec 5, 2018
@ghost ghost added the review label Dec 5, 2018
@faustbrian faustbrian merged commit aee3d00 into master Dec 5, 2018
@ghost ghost removed the review label Dec 5, 2018
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