[FLINK-11447][table] Deprecate new Table(TableEnvironment, String)

[dianfu]

What is the purpose of the change

This pull request deprecates the constructor new Table(TableEnvironment, String)

Brief change log

• Deprecates new Table(TableEnvironment, String)
• Introduces Table.lateralJoin, Table.leftOuterLateralJoin
• Move the implicit of 'TableFunction to Table' to ExpressionDsl as otherwise this implicit always has the highest priority and TableFunction will always be converted to table which is not expected for joinLateral/leftOuterJoinLateral

Verifying this change

This change added tests and can be verified as follows:

(example:)

• Added tests CorrelateStringExpressionTest.testCorrelateJoinsWithJoinLateral

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: (yes)
• The runtime per-record code paths (performance sensitive): (no)
• Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
• The S3 file system connector: (no)

Documentation

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

[flinkbot]

Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community
to review your pull request. We will use this comment to track the progress of the review.

Review Progress

• ✅ 1. The [description] looks good.
  • Approved by @twalthr [PMC]
• ✅ 2. There is [consensus] that the contribution should go into to Flink.
  • Approved by @twalthr [PMC]
• ❔ 3. Needs [attention] from.
• ✅ 4. The [architecture] is sound.
  • Approved by @twalthr [PMC]
• ✅ 5. Overall code [quality] is good.
  • Approved by @twalthr [PMC]

Please see the Pull Request Review Guide for a full explanation of the review process.

Bot commands

The @flinkbot bot supports the following commands:

• @flinkbot approve description to approve the 1st aspect (similarly, it also supports the consensus, architecture and quality keywords)
• @flinkbot approve all to approve all aspects
• @flinkbot attention @username1 [@username2 ..] to require somebody's attention
• @flinkbot disapprove architecture to remove an approval

[dianfu]

@twalthr @dawidwys Could you help to take a look at this PR when it's convenient for you? Thanks in advance.

[twalthr]

Thank you @dianfu. I did a quick scan over the changes. Can you search again for outdated website docs or Javadocs. For example, the docs of {{org.apache.flink.table.api.functions.TableFunction}} still need an update. I will do a full review soon. Thanks.

[twalthr]

@flinkbot approve description
@flinkbot approve consensus

[dianfu]

@twalthr Thanks a lot for the remind. I have updated the documentation.

[twalthr]

@flinkbot approve all

Thanks for the update @dianfu. I spotted some additional usages but I will fix them while merging. In general the changes look good to me. I will do the final polishing and merge this.