Didn't fully support sync view while remove view from builtin skip DDLs

[lance6716]

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. What did you do? If possible, provide a recipe for reproducing the error.

create view in upstream

2. What did you expect to see?

3. What did you see instead?

error 11007 unknown type DDL

4. Versions of the cluster

   • DM version (run dmctl -V or dm-worker -V or dm-master -V):

     master

[lance6716]

in #1105 we allow syncing of CREATE VIEW, but didn't add an integration test (unit test didn't expose the problem, which maybe SplitDDL can't handle VIEW)

[lance6716]

since we didn't dump&load view, how about still filter out VIEW in sync 😂

[csuzhangxc]

since we didn't dump&load view, how about still filter out VIEW in sync 😂

or add support for dump&load view?

[lance6716]

since we didn't dump&load view, how about still filter out VIEW in sync 😂

or add support for dump&load view?

OK, will learn from pingcap/tidb-lightning#417. I'm not sure this could be finished in coming release 😂

[lichunzhu]

Just load views after all the tables have been loaded. Dumpling has already supported dumping views like mydumper pingcap/dumpling#158

[cyliu0]

A community user is questioning about dm supporting view yesterday. So this could be a customer issue in some cases.

[glkappe]

@lance6716 How is the progress of this issue?

[lance6716]

@lance6716 How is the progress of this issue?

above this comment is a pull request that fixes this issue. if it gets reviewed quickly, this could be supported in nightly this week (but not well tested).

[lance6716]

block syncing of VIEW in #1331 and will support it later. changed to feature-request

[lance6716]

repalced by #1622