Skip to content
This repository was archived by the owner on Jan 12, 2026. It is now read-only.

[BitSail][Connector]add oss source connector#467

Open
CodingGPT wants to merge 2 commits into
bytedance:masterfrom
CodingGPT:add-oss-source-connector
Open

[BitSail][Connector]add oss source connector#467
CodingGPT wants to merge 2 commits into
bytedance:masterfrom
CodingGPT:add-oss-source-connector

Conversation

@CodingGPT

Copy link
Copy Markdown

Prepare a Pull Request

Motivation

add new source connector

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

Comment thread bitsail-connectors/connector-oss/pom.xml Outdated
Comment thread bitsail-connectors/connector-oss/pom.xml Outdated
wzhero1

This comment was marked as outdated.

@wzhero1

wzhero1 commented Apr 11, 2023

Copy link
Copy Markdown

The resource folder and associated files are missing

@wzhero1

wzhero1 commented Apr 11, 2023

Copy link
Copy Markdown

OssConf and HadoopConf can be merged into one file, just OssConf

@wzhero1 wzhero1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

new suggestions

@CodingGPT

Copy link
Copy Markdown
Author

The resource folder and associated files are missing
resource files already added and OssConf and HadoopConf was merged

@CodingGPT CodingGPT requested a review from wzhero1 April 19, 2023 15:42
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