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

horizon/db2/history: optimize account queries for payments and operations #829

Merged
merged 2 commits into from
Jan 24, 2019

Conversation

howardtw
Copy link
Contributor

@howardtw howardtw commented Jan 23, 2019

This PR is the result of #811 (comment). This is basically a modified version of #439

Close #515

@bartekn
Copy link
Contributor

bartekn commented Jan 23, 2019

LGTM but can you add a simple test to check if the output SQL query is using the correct field?

@howardtw
Copy link
Contributor Author

howardtw commented Jan 23, 2019

@bartekn A test has been added db972d9

@bartekn bartekn merged commit 11d7ac7 into stellar:master Jan 24, 2019
@howardtw howardtw deleted the better-query branch January 29, 2019 19:25
oryband added a commit to kinecosystem/go that referenced this pull request Feb 17, 2019
* tag 'horizon-v0.16.0':
  horizon: 0.16.0 CHANGELOG (stellar#851)
  Allow smooth migration for failed transactions column (stellar#847)
  Clean up the redundancy in the Horizon command line option handling (stellar#834)
  horizon/actions: fix a bug in `GetPageQuery` and `GetAddress` (stellar#846)
  remove JSON
  horizon/actions: check hash before sending an event to the stream of orderbook (stellar#828)
  Deprecated check (stellar#838)
  Report right protocol version in horizon root path (stellar#783)
  fix migration ordering (stellar#821)
  Add failed transactions to ledger resource (stellar#756)
  Ignore "0" trades effects (stellar#791)
  Add package index to README (stellar#831)
  horizon/db2/history: optimize account queries for payments and operations (stellar#829)
  Update "gap detected" message (stellar#830)
  Update setTickInProgress comment (stellar#824)
  Bump postgres version for Travis (stellar#822)
  fix wrong effect of a circular path payment (stellar#807)
  Update README.md (stellar#820)
  Fix build badge (stellar#816)
  Require migration to run Horizon version successfully (fixes stellar#778) (stellar#808)
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.

2 participants