[Bug] [Seatunnel-web] Error when conditional column is not used in SE…

[BilwaST]

…LECT clause

Purpose of this pull request

Check list

• Code changed are covered with tests, or it does not need tests for reason:
• If any new Jar binary package adding in your PR, please add License Notice according
  New License Guide
• If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs

[bilst-visa]

Test Report:

job profile content with my changes:

[arshadmohammad]

LGTM +1

[Hisoka-X]

I think the better place to do this thing is in

seatunnel-web/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/thirdparty/datasource/impl/BaseJdbcDataSourceConfigSwitcher.java

Line 141 in d862b61

connectorConfig =

The place fine tuning source config of jdbc.

[BilwaST]

Hi @Hisoka-X

I did not add it here as not all DataSourceConfigSwitcher extends this BaseJdbcDataSourceConfigSwitcher class. For example HiveDataSourceConfigSwitcher.

[Hisoka-X]

Hi @Hisoka-X

I did not add it here as not all DataSourceConfigSwitcher extends this BaseJdbcDataSourceConfigSwitcher class. For example HiveDataSourceConfigSwitcher.

But the where_condition only worked on jdbc connector, so change it in BaseJdbcDataSourceConfigSwitcher is more reasonable.

[arshadmohammad]

Thanks @Hisoka-X for suggesting the change.
yes, it makes better sense to put the changes in BaseJdbcDataSourceConfigSwitcher.
@BilwaST can you raise another addendum PR

[BilwaST]

Thank you @Hisoka-X @arshadmohammad for the feedback. I have raised another PR #239