[File system] Add Google Cloud Storage integration #631
[File system] Add Google Cloud Storage integration #631luoyuxia merged 2 commits intoapache:mainfrom
Conversation
|
Will add integration tests. Putting temporary to draft |
|
Added an integration test based on Since the service account key is in a file form and not in a string form I see two options.
In the pr I followed the second approach. |
|
@luoyuxia could you help to review it? |
|
@gkatzioura , could you check how Flink tests the gs-filesystem in CI? |
|
Hi @wuchong , after checking there are not tests for the actual filesystem integration. |
|
Thank you @gkatzioura , please ping me when you have finished the mock server. |
|
Hi @wuchong & @luoyuxia |
|
checkstyle are failed |
|
@luoyuxia could you help to review this PR? |
Sure, now I got some free time. I'll help review these days. |
luoyuxia
left a comment
There was a problem hiding this comment.
@gkatzioura Thanks for the greate work! Left some comments.
|
Hi @gkatzioura , you can "request review" again when you have addressed all the comments, so that the reviewer can get the notification in time. |
luoyuxia
left a comment
There was a problem hiding this comment.
@gkatzioura Could you please rebase main branch and add licencese since currently licences check is enabeld.
| import static org.assertj.core.api.Assertions.assertThat; | ||
| import static org.assertj.core.api.AssertionsForClassTypes.assertThatThrownBy; | ||
|
|
||
| class GSFileSystemPluginTest { |
There was a problem hiding this comment.
Please don't forget this comment.
694a42f to
c465149
Compare
|
@gkatzioura Thanks for the great work. I'll have a final check. Hope to merge in later days |
luoyuxia
left a comment
There was a problem hiding this comment.
@gkatzioura Thanks for the work. LGTM! I append a commit to add licenses and refine test. Please take a look. If you have no further comments, I'll merge later
0b8f285 to
bec6304
Compare
Purpose
Linked issue: close #630
Brief change log
Added the GSFileSystem based on the given Hadoop Google Cloud Storage file system
Tests
API and Format
No
Documentation
It adds support for the Google Cloud Storage Filesystem