Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

feat: basic implement #4

Closed
wants to merge 2 commits into from

Conversation

timber-joan
Copy link

Implement basic Storager functions.
Update related confs.

Passed all go test.
go-test-res jpg

@Xuanwo
Copy link
Contributor

Xuanwo commented Jun 7, 2021

First of all, thank you very much for your contribution!

Are you a candidate for the OSPP? And I can't find your applications on our forum.

@timber-joan
Copy link
Author

First of all, thank you very much for your contribution!

Are you a candidate for the OSPP? And I can't find your applications on our forum.

Sorry for the late reply.
I haven't seen that docs before, so I didn't submit my application on that forum.

I had posted: proposal .

service.toml Outdated Show resolved Hide resolved
service.toml Outdated Show resolved Hide resolved
service.toml Outdated
optional = ["offset", "io_callback", "size", "file_version"]

[namespace.storage.op.stat]
optional = ["file_version", "object_type"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto.

And we have ObjectMode, object_type is not needed.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't found ObjectMode in stat ops, where should I found?
I separate these two types because different object type need different operations in minio-client, and stat to a folder object may cost more time.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GSP-93 has been approved, we can use ObjectMode in stat now.

minio doesn't have native support for DirObject, so we need to simulate it. Let's implement in another PR.

For current PR, It's OK to implement just like no dir object exists.
And in next PR, we need to:

  • Implement CreateDir with simulate = true (described in GSP-87)
  • Implement Stat / Delete a dir object.

NOTICE: go-storage requires zero-cost abstraction. so we can't use ListObjects API in stat.

service.toml Outdated Show resolved Hide resolved
service.toml Outdated Show resolved Hide resolved
service.toml Outdated Show resolved Hide resolved
storage.go Outdated Show resolved Hide resolved
storage.go Outdated Show resolved Hide resolved
storage.go Outdated Show resolved Hide resolved
storage.go Outdated Show resolved Hide resolved
@Xuanwo
Copy link
Contributor

Xuanwo commented Jun 9, 2021

In order to make builds pass, we need to change go 1.16 to go 1.15 in go.mod

@Xuanwo
Copy link
Contributor

Xuanwo commented Jul 14, 2021

@timber-joan Thank you very much for your hard work and dedication, but PR #6 has been merged, so we had to close this PR. Hope to see your contribution again sometime!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants