[
https://issues.apache.org/jira/browse/FLINK-31380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17701912#comment-17701912
]
Ran Tao edited comment on FLINK-31380 at 3/17/23 5:24 PM:
----------------------------------------------------------
Hi, [~jark] . Currently there are three ways to support these auxiliary sql
statements.
1.change every operation to support like/ilike or from/in, but future syntax we
need to continue change them. (one-by-one way).
2.add a common abstract class to do this, but future change we need to change
abstract class either.(abstract class way)
3.decorator model, add concrete decorator to support a certain syntax. Its
benefits are very scalable and loosely coupled. currently the pr was
implemented in this way. (decorators)
I'm wondering if I've overcomplicated the problem(because these classes are not
@Public). WDYT?
was (Author: lemonjing):
Hi, [~jark] . Currently there are three ways to support these auxiliary sql
statements.
1.change every operation to support like/ilike or from/in, but future syntax we
need to continue change them. (one-by-one way).
2.add a common abstract class to do this, but future change we need to change
abstract class either.(abstract class way)
3.decorator model, add concrete decorator to support a certain syntax. Its
benefits are very scalable and loosely coupled. currently the pr was
implemented in this way.
I'm wondering if I've overcomplicated the problem(because these classes are not
@Public). WDYT?
> Support enhanced show catalogs syntax
> -------------------------------------
>
> Key: FLINK-31380
> URL: https://issues.apache.org/jira/browse/FLINK-31380
> Project: Flink
> Issue Type: Sub-task
> Components: Table SQL / API, Table SQL / Planner
> Reporter: Ran Tao
> Assignee: Ran Tao
> Priority: Major
> Labels: pull-request-available
>
> As FLIP discussed. We will support new syntax for some show operations.
> To avoid bloat, this ticket supports ShowCatalogs.
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
