File info fix #735

Closed
hackrush01 wants to merge 4 commits from file_info_fix into master
hackrush01 commented 2017-11-13 19:22:32 +01:00 (Migrated from github.com)
No description provided.
kauffj (Migrated from github.com) requested changes 2017-11-15 22:56:36 +01:00
kauffj (Migrated from github.com) left a comment

Looks close, biggest question is if there is a slightly better way to do this that doesn't require carrying around and translating between both outpoints and sd_hashes.

Looks close, biggest question is if there is a slightly better way to do this that doesn't require carrying around and translating between both outpoints and sd_hashes.
kauffj (Migrated from github.com) commented 2017-11-15 22:55:39 +01:00

Can this be re-written to only use sd_hash rather than both outpoint and sd_hash?

Can this be re-written to only use `sd_hash` rather than both `outpoint` and `sd_hash`?
kauffj (Migrated from github.com) commented 2017-11-15 22:53:53 +01:00

Can you help me understand why this is necessary? When do I have an outpoint that I need to translated into an sd_hash? Can we just stored sd_hashes in that place instead?

Can you help me understand why this is necessary? When do I have an outpoint that I need to translated into an sd_hash? Can we just stored sd_hashes in that place instead?
kauffj (Migrated from github.com) commented 2017-11-15 22:54:02 +01:00

ahem

*ahem*
kauffj (Migrated from github.com) commented 2017-11-15 22:51:52 +01:00

.source.source? Guessing this is correct, but please tell me who to complain to :P

`.source.source`? Guessing this is correct, but please tell me who to complain to :P
kauffj (Migrated from github.com) commented 2017-11-15 22:50:04 +01:00

sdHashes

`sdHashes`
hackrush01 (Migrated from github.com) reviewed 2017-11-17 19:27:36 +01:00
hackrush01 (Migrated from github.com) commented 2017-11-17 19:27:36 +01:00

TL;DR I'm not sure.
It went a bit too deep inside lbry.js and it seemed to be intertwined with other stuff(such as pending publishes, dummy claims). Overwhelmed by the complications involved I couldn't muster up enough courage to mess with it.

TL;DR I'm not sure. It went a bit too deep inside `lbry.js` and it seemed to be intertwined with other stuff(such as pending publishes, dummy claims). Overwhelmed by the complications involved I couldn't muster up enough courage to mess with it.
hackrush01 (Migrated from github.com) reviewed 2017-11-17 19:29:25 +01:00
hackrush01 (Migrated from github.com) commented 2017-11-17 19:29:25 +01:00

I think it is necessary due to what happens when I have a pending publish(related to https://github.com/lbryio/lbry-app/pull/735#discussion_r151756127 probably).

I think it is necessary due to what happens when I have a pending publish(related to https://github.com/lbryio/lbry-app/pull/735#discussion_r151756127 probably).
hackrush01 (Migrated from github.com) reviewed 2017-11-17 19:29:44 +01:00
hackrush01 (Migrated from github.com) commented 2017-11-17 19:29:44 +01:00

Oops!

Oops!
hackrush01 (Migrated from github.com) reviewed 2017-11-17 19:31:16 +01:00
hackrush01 (Migrated from github.com) commented 2017-11-17 19:31:16 +01:00

The schema/lbryum/daemon guys.

The schema/lbryum/daemon guys.
hackrush01 (Migrated from github.com) reviewed 2017-11-17 19:31:29 +01:00
hackrush01 (Migrated from github.com) commented 2017-11-17 19:31:29 +01:00

Again, Oops!

Again, Oops!
tzarebczan commented 2017-11-20 17:51:38 +01:00 (Migrated from github.com)
Some scenarios to test alongside these changes: https://github.com/lbryio/lbry/issues/951 https://github.com/lbryio/lbry/issues/944 https://github.com/lbryio/lbry-app/issues/600
kauffj commented 2017-11-24 21:09:35 +01:00 (Migrated from github.com)

I'm not planning to merge this without reducing the amount of values carried around from 2 to 1. If sd_hash is the new key for file data instead of outpoint, let's eliminate the usage of outpoint basically everywhere.

I'm not planning to merge this without reducing the amount of values carried around from 2 to 1. If `sd_hash` is the new key for file data instead of `outpoint`, let's eliminate the usage of `outpoint` basically everywhere.
hackrush01 commented 2017-11-28 16:57:47 +01:00 (Migrated from github.com)

I'll look into it and rebase it on master soon.

I'll look into it and rebase it on master soon.
kauffj commented 2017-11-28 17:00:35 +01:00 (Migrated from github.com)

@hackrush01 I've actually made a bunch of progress on this separately in the flow_and_file_list branch (I used it as my test for writing code typed with Flow). I'll finish this.

@hackrush01 I've actually made a bunch of progress on this separately in the `flow_and_file_list` branch (I used it as my test for writing code typed with Flow). I'll finish this.
hackrush01 commented 2017-11-28 17:18:21 +01:00 (Migrated from github.com)

Ok

Ok
lyoshenka commented 2017-12-12 20:01:33 +01:00 (Migrated from github.com)

blocked by lbryio/lbry#1025

blocked by lbryio/lbry#1025
kauffj commented 2018-02-28 17:05:19 +01:00 (Migrated from github.com)

This is no longer necessary, @liamcardenas has handled this.

This is no longer necessary, @liamcardenas has handled this.

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#735
No description provided.