search

qri-io / rfcs

Request For Comments (RFCs) documenting changes to Qri
MIT License
12 stars 6 forks source link

rfc: load_dataset #38

Closed b5 closed 5 years ago

b5 commented 5 years ago

I've had some time to live with starlark now, and a number of use cases that have cropped up where depending on a dataset with qri.load_dataset has been problematic. I initially thought this was a permissions problem, describing the issue as required "permission escalation" to allow access to datasets during the download special function.

While working through this RFC, I've come to believe the problem is the way we declare dataset dependencies. This RFC aims to clear that up while pushing the problem of access control down the road, but still make some progress on how access control will integrate with this RFC.

b5 commented 5 years ago

Thanks for the solid review team, I've worked in requested changes. Can I get a vote on ratifying this?

b5 commented 5 years ago

@ramfox can we get a review on this? thx!