lnd icon indicating copy to clipboard operation
lnd copied to clipboard

Add pagination options to listchaintxns

Open 34ro opened this issue 7 years ago • 13 comments

This PR adds pagination to listchaintxns. It prevents client application from out of memory when the number of transactions increases.

The new options are offset and limit. Offset is the numer of transactions to skip. Limit is max numer of transactions. RPC also has same request field.

./lncli listchaintxns --offset=100 --limit=50

All merged transactions and unmerged transactions are sort by timestamp. I think sorting by timestamp is the best because it makes order invariable.

34ro avatar Jan 06 '18 16:01 34ro

Also this will need to be rebased to the latest master.

Roasbeef avatar Apr 06 '18 03:04 Roasbeef

rebased

34ro avatar Apr 08 '18 13:04 34ro

Thanks for the rebase, but none of the review comments were addressed.

Roasbeef avatar Apr 18 '18 00:04 Roasbeef

Please wait for a little longer.

34ro avatar Apr 18 '18 05:04 34ro

rebased. lnrpc/gen_protos.sh made some difference.

@Roasbeef How can I execute test with travice?

34ro avatar Jul 22 '18 08:07 34ro

@34ro Check if your account has been flagged on Travis. if it has you'll have to e-mail them to un-flag it :/

halseth avatar Jul 24 '18 07:07 halseth

Ping @wpaulino.

Roasbeef avatar Aug 10 '18 04:08 Roasbeef

rebased

34ro avatar Aug 13 '18 14:08 34ro

rebased

34ro avatar Aug 20 '18 01:08 34ro

rebased

34ro avatar Aug 26 '18 02:08 34ro

Looks like this essentially address https://github.com/lightningnetwork/lnd/issues/4719 right?

mrfelton avatar Jan 14 '21 19:01 mrfelton

@mrfelton yep, though I think we may just need to remake this PR since it's so old, and we have a new internal pattern we use for adding pagination to existing RPCs.

Roasbeef avatar Jan 20 '21 23:01 Roasbeef

@34ro, remember to re-request review from reviewers when ready

lightninglabs-deploy avatar Sep 13 '22 07:09 lightninglabs-deploy

@34ro, remember to re-request review from reviewers when ready

lightninglabs-deploy avatar Nov 15 '22 12:11 lightninglabs-deploy

I'll take a look at this given the latest request from a user: https://github.com/lightningnetwork/lnd/issues/4719#issuecomment-1447636264

hieblmi avatar Jul 25 '23 11:07 hieblmi

@34ro, remember to re-request review from reviewers when ready

lightninglabs-deploy avatar Jan 16 '24 22:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 00:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 01:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 02:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 04:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 05:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 06:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 07:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 08:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 09:01 lightninglabs-deploy

Closing due to inactivity

lightninglabs-deploy avatar Jan 22 '24 10:01 lightninglabs-deploy