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

Remove Height Flag from CLI Queries #3883

Closed
4 tasks
KenianShi opened this issue Mar 14, 2019 · 2 comments · Fixed by #3910
Closed
4 tasks

Remove Height Flag from CLI Queries #3883

KenianShi opened this issue Mar 14, 2019 · 2 comments · Fixed by #3910

Comments

@KenianShi
Copy link

KenianShi commented Mar 14, 2019

Summary of Bug

my account does not create until after 1000 blocks were created,but i can query it's account when i use --height 1.i used
'gaiacli query account --height 1 cosmos1mt3zl9hveal9rnqd5w5sr8v9t8q5y7ag3wjmk4',

it returned "Account:
Address: cosmos1mt3zl9hveal9rnqd5w5sr8v9t8q5y7ag3wjmk4
Pubkey: cosmospub1addwnpepqwer0m3xpzjee4aancqtxdy5ydewt50heds39hsxtrlwvkjh4qhaxznqqu4
Coins: 999881skn,9032772stake
AccountNumber: 1
Sequence: 11"

Steps to Reproduce

when you send some token from A to B,query account of A and B,add --height int. int is the block number before the send tx and after the send tx.you will find the balance is same


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@cwgoes
Copy link
Contributor

cwgoes commented Mar 14, 2019

We should remove this option from the CLI. Height queries do not work at present, and will not generally work unless the full node servicing the query stores all past state versions.

@alexanderbez
Copy link
Contributor

Agreed on removing the height option.

@alexanderbez alexanderbez changed the title --height does not work Remove height flag from CLI queries Mar 14, 2019
@alexanderbez alexanderbez changed the title Remove height flag from CLI queries Remove Height Flag from CLI Queries Mar 14, 2019
alessio pushed a commit that referenced this issue Mar 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants