[FLINK-26618][sql-client] Fix Remove-Jar statement not effective

[link3280]

What is the purpose of the change

Currently, remove jar statement doesn't remove the corresponding jars in pipeline.jars, which should be fixed.

Brief change log

• Replacing session dependencies instead of merging them into the new dependencies in remove-jar statements.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

• Dependencies (does it add or upgrade a dependency): (no)
• The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
• The serializers: (no)
• The runtime per-record code paths (performance sensitive): (no)
• Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
• The S3 file system connector: (no)

Documentation

• Does this pull request introduce a new feature? (no)
• If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

[flinkbot]

CI report:

• bf8fac5 Azure: SUCCESS

Bot commands

The @flinkbot bot supports the following commands:

• @flinkbot run azure re-run the last Azure build

[link3280]

CI failures are irrelevant, retrigger the pipeline.

[link3280]

@flinkbot run azure

[link3280]

@flinkbot run azure

[fsk119]

Thanks for your great work.

Could you also add a case in the CliClientITCase? You can take a look at the file set.q[1] for more details.

[1] https://github.com/apache/flink/blob/master/flink-table/flink-sql-client/src/test/resources/sql/set.q#L128

[fsk119]

Thanks for your update. Leave one minor suggestion.

[fsk119]

cc @leonardBang @wuchong

[fsk119]

Thanks for your update. LGTM

[link3280]

Thanks a lot for your review! @fsk119

[leonardBang]

+1

[leonardBang]

@flinkbot run azure

[leonardBang]

@link3280 Could you also open PRs for branch release-1.14 and release-1.15 ?

[link3280]

@leonardBang Sure.

[leonardBang]

@flinkbot run azure

[link3280]

@leonardBang Opened 2 backport PR #19132 #19133

[leonardBang]

@leonardBang Opened 2 backport PR #19132 #19133

Nice work！We need rebase this PR to latest master to pass the e2e tests, the e2e tests failed today and has been fixed in
cbf2ab0

[link3280]

@leonardBang Done. Updating the other two PRs.

[link3280]

@flinkbot run azure

[link3280]

@flinkbot run azure

[link3280]

@leonardBang CI finally turns green. Could you merge this PR if possible? Thanks.

[leonardBang]

Thanks @link3280 for the continuous work, the Github infrastructure broken recently which influenced many project 😂.