Skip to content

[fix-#11691] Add lz4-java in clickhouse datasource.#11697

Merged
ruanwenjun merged 4 commits intoapache:devfrom
fuchanghai:fix-11691
Sep 5, 2022
Merged

[fix-#11691] Add lz4-java in clickhouse datasource.#11697
ruanwenjun merged 4 commits intoapache:devfrom
fuchanghai:fix-11691

Conversation

@fuchanghai
Copy link
Copy Markdown
Member

Purpose of the pull request

Brief change log

lack of dependencies

Verify this pull request

This pull request is code cleanup without any test coverage.

this close #11691

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 30, 2022

Codecov Report

Merging #11697 (4d40806) into dev (4a37345) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #11697      +/-   ##
============================================
- Coverage     39.81%   39.80%   -0.02%     
+ Complexity     4727     4723       -4     
============================================
  Files          1006     1006              
  Lines         37936    37936              
  Branches       4236     4237       +1     
============================================
- Hits          15106    15101       -5     
- Misses        21224    21227       +3     
- Partials       1606     1608       +2     
Impacted Files Coverage Δ
...er/master/dispatch/host/assign/RandomSelector.java 77.77% <0.00%> (-5.56%) ⬇️
...erver/master/processor/queue/TaskEventService.java 75.00% <0.00%> (-5.36%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 52.08% <0.00%> (-0.70%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@SbloodyS SbloodyS added the bug Something isn't working label Aug 30, 2022
@SbloodyS SbloodyS added this to the 3.0.1 milestone Aug 30, 2022
@fuchanghai fuchanghai closed this Aug 30, 2022
@fuchanghai fuchanghai reopened this Aug 30, 2022
@SbloodyS SbloodyS added the dependencies Pull requests that update a dependency file label Aug 30, 2022
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@fuchanghai
Copy link
Copy Markdown
Member Author

@kezhenxu94 PTAL

@ruanwenjun ruanwenjun changed the title [fix-#11691] lack of dependencies [fix-#11691] Add lz4-java in clickhouse datasource. Sep 5, 2022
@ruanwenjun ruanwenjun merged commit 554562b into apache:dev Sep 5, 2022
@fuchanghai fuchanghai deleted the fix-11691 branch September 6, 2022 05:42
fengjian1129 pushed a commit to fengjian1129/dolphinscheduler that referenced this pull request Sep 13, 2022
caishunfeng pushed a commit to caishunfeng/dolphinscheduler that referenced this pull request Sep 17, 2022
caishunfeng added a commit that referenced this pull request Sep 17, 2022
* [Cherry-pick-3.0.1][fix-#11691] Add lz4-java in clickhouse datasource. #11697

* [Cherry-pick-3.0.1][fix-#11689] change replaceAll to replace in method of tryExecuteSqlResolveColumnNames #11696

* [fix-11378] fix API-Server release response message typo (#11400)

* [fix-11295] remove duplicate classes

* [fix-11378] fix response message error

Co-authored-by: fuchanghai <changhai.fu@marketingforce.com>

Co-authored-by: fuchanghai <33984497+fuchanghai@users.noreply.github.com>
Co-authored-by: fuchanghai <changhai.fu@marketingforce.com>
@caishunfeng caishunfeng added the release cherry-pick Mark this issue/PR had cherry-pick for release version label Sep 17, 2022
xdu-chenrj pushed a commit to xdu-chenrj/dolphinscheduler that referenced this pull request Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend bug Something isn't working dependencies Pull requests that update a dependency file release cherry-pick Mark this issue/PR had cherry-pick for release version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [dolphinscheduler-datasource-clickhouse] lack of dependencies

7 participants