search

siphonelee / SuiS3

SuiS3 is a tool that presents an AWS S3-style CLI, facilitating the effective management of Walrus' flat data storage in a structured and meaningful hierarchy, through the related metadata stored on Sui.
3 stars 0 forks source link

Consider removing suis3:// schema from the paths #1

Open kkomelin opened 2 days ago

kkomelin commented 2 days ago

Hi Calvin,

Really promising project! Great job!

I'd like to ask why do you need that shema prefixes "suis3://" if you're already "logged in" to suis3?

suis3 > ls suis3://bucket1
siphonelee commented 1 day ago

That's a good question @kkomelin. Actually the previous style was no "suis3" prefix. But I finally changed the style after considering the following factors:

  1. to keep consistent with AWS S3 CLI style like this: aws s3 cp /tmp/foo s3://bucket/ ;
  2. Besides interative shell mode, it was forgotten to mention in README that SuiS3 also supports "single shot" mode, just like the example above. It would appear more naturally to have the prefix in this mode.
kkomelin commented 1 day ago

Ah, I see now @siphonelee . Makes sense. Thanks.

I feel that they will be positioning Walrus as a separate product (I might be wrong), so I think it would also make sense to change the schema to wal:// or create such an alias for suis3://.

kkomelin commented 1 day ago

Feel free to close the issue when ready. Thanks