Skip to content

[feature][datasource] Main add datasource-pulsar#145

Open
wachoo wants to merge 3 commits intoapache:mainfrom
wachoo:dev
Open

[feature][datasource] Main add datasource-pulsar#145
wachoo wants to merge 3 commits intoapache:mainfrom
wachoo:dev

Conversation

@wachoo
Copy link
Contributor

@wachoo wachoo commented Nov 2, 2023

Purpose of this pull request

Check list

@wachoo wachoo changed the title add datasource-pulsar [feature][datasource] Main add datasource-pulsar Nov 2, 2023
@wachoo wachoo force-pushed the dev branch 5 times, most recently from 6a623de to 2766026 Compare November 2, 2023 11:07
@wachoo wachoo force-pushed the dev branch 3 times, most recently from 7aae9cd to c4632ff Compare November 10, 2023 09:26
EricJoy2048
EricJoy2048 previously approved these changes Nov 13, 2023
<artifactId>mysql-connector-java</artifactId>
<version>${mysql.version}</version>
<scope>test</scope>
<scope>provided</scope>
Copy link
Member

Choose a reason for hiding this comment

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

mysql-connector-java license is GPL, So, must use scope test here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thx,I got it.

Copy link
Contributor

@davidzollo davidzollo Dec 27, 2023

Choose a reason for hiding this comment

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

I help update the pom.xml

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thx, Merging is still blocked.

davidzollo
davidzollo previously approved these changes Dec 28, 2023
Copy link
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

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

+1
LGTM

@wachoo
Copy link
Contributor Author

wachoo commented Mar 9, 2024

+1 LGTM

haha, thx

@wachoo wachoo requested review from EricJoy2048 and davidzollo June 20, 2024 04:28
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.

3 participants