Skip to content

[#2640] feat(docs): Add the doc for gvfs#2791

Merged
jerryshao merged 10 commits into
apache:mainfrom
xloya:add-doc-for-gvfs
Apr 7, 2024
Merged

[#2640] feat(docs): Add the doc for gvfs#2791
jerryshao merged 10 commits into
apache:mainfrom
xloya:add-doc-for-gvfs

Conversation

@xloya

@xloya xloya commented Apr 3, 2024

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add gvfs usage documentation. Currently, three usages in hadoop shell, java code and spark program are introduced.

Why are the changes needed?

Fix: #2640

Comment thread docs/how-to-use-gvfs.md
@qqqttt123

qqqttt123 commented Apr 3, 2024

Copy link
Copy Markdown
Contributor

Could you polish the document according to https://github.com/datastrato/gravitino/pull/2781?

Comment thread docs/how-to-use-gvfs.md Outdated
Comment thread docs/how-to-use-gvfs.md Outdated
Comment thread docs/how-to-use-gvfs.md Outdated
@xloya

xloya commented Apr 3, 2024

Copy link
Copy Markdown
Contributor Author

Could you polish the document according to #2781?

Yeah, I am only open the PR and will make some modifications based on this document.

Comment thread docs/how-to-use-gvfs.md Outdated
Comment thread docs/how-to-use-gvfs.md
@jerryshao

Copy link
Copy Markdown
Contributor

Hi @xloya thanks a lot for your work. I can help you to add more content to this doc.

Comment thread docs/how-to-use-gvfs.md
Comment thread docs/how-to-use-gvfs.md
Comment thread docs/how-to-use-gvfs.md Outdated
The following mainly introduces how to use `Gravitino Virtual File System` to operate fileset which mapping to HDFS.


## Prerequisites

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What about safety of Gvfs?

@xloya

xloya commented Apr 3, 2024

Copy link
Copy Markdown
Contributor Author

Hi @xloya thanks a lot for your work. I can help you to add more content to this doc.

Sure, please see what needs to be added.

@jerryshao

Copy link
Copy Markdown
Contributor

@xloya @coolderli @qqqttt123 I slightly update the doc, would you please help to review?

@jerryshao jerryshao closed this Apr 7, 2024
@jerryshao jerryshao reopened this Apr 7, 2024
@jerryshao

Copy link
Copy Markdown
Contributor

I think we should also add Tensorflow support in this doc, WDYT @coolderli ?

@coolderli

Copy link
Copy Markdown
Contributor

I think we should also add Tensorflow support in this doc, WDYT @coolderli ?

@jerryshao Yes. I think I can add a commit about tensorflow. And you can improve it.

Comment thread docs/how-to-use-gvfs.md Outdated
```


### Use GVFS with Tensorflow.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@jerryshao I have added docs about TensorFlow. Please take a look and improve it if needed.

Comment thread docs/how-to-use-gvfs.md Outdated
Comment thread docs/how-to-use-gvfs.md Outdated
@qqqttt123

Copy link
Copy Markdown
Contributor

Others are ok for me. Thanks for your contribution.

Comment thread docs/how-to-use-gvfs.md Outdated
xiaojiebao and others added 2 commits April 7, 2024 18:45
Comment thread docs/how-to-use-gvfs.md Outdated
Comment thread docs/how-to-use-gvfs.md
@jerryshao

Copy link
Copy Markdown
Contributor

I've renamed the package from hadoop3 to hadoop, WDYT? @xloya @coolderli

@qqqttt123

Copy link
Copy Markdown
Contributor

I've renamed the package from hadoop3 to hadoop, WDYT? @xloya @coolderli

+1 for me. Hadoop is better than hadoop3.

@qqqttt123

Copy link
Copy Markdown
Contributor

LGTM.

@xloya

xloya commented Apr 7, 2024

Copy link
Copy Markdown
Contributor Author

I've renamed the package from hadoop3 to hadoop, WDYT? @xloya @coolderli

It's okay.

@coolderli

Copy link
Copy Markdown
Contributor

I've renamed the package from hadoop3 to hadoop, WDYT? @xloya @coolderli

+1 for me. Hadoop is better than hadoop3.

+1 for me.

@jerryshao jerryshao merged commit b8f04c5 into apache:main Apr 7, 2024
unknowntpo pushed a commit to unknowntpo/gravitino that referenced this pull request Apr 9, 2024
### What changes were proposed in this pull request?

Add gvfs usage documentation. Currently, three usages in `hadoop shell`,
`java code` and `spark program` are introduced.

### Why are the changes needed?

Fix: apache#2640

---------

Co-authored-by: xiaojiebao <xiaojiebao@xiaomi.com>
Co-authored-by: Jerry Shao <jerryshao@datastrato.com>
Co-authored-by: lipeidian <lipeidian@qq.com>
@xloya xloya deleted the add-doc-for-gvfs branch June 20, 2024 01:43
danhuawang pushed a commit to danhuawang/gravitino that referenced this pull request Jun 9, 2026
### What changes were proposed in this pull request?

Add gvfs usage documentation. Currently, three usages in `hadoop shell`,
`java code` and `spark program` are introduced.

### Why are the changes needed?

Fix: apache#2640

---------

Co-authored-by: xiaojiebao <xiaojiebao@xiaomi.com>
Co-authored-by: Jerry Shao <jerryshao@datastrato.com>
Co-authored-by: lipeidian <lipeidian@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Subtask] Add the gvfs user doc

4 participants