Skip to content
This repository has been archived by the owner on Nov 24, 2023. It is now read-only.

Not working when use uppercase event in binlog event filter #181

Closed
kalvinzhang opened this issue Jun 21, 2019 · 2 comments
Closed

Not working when use uppercase event in binlog event filter #181

kalvinzhang opened this issue Jun 21, 2019 · 2 comments
Labels
type/bug This issue is a bug report

Comments

@kalvinzhang
Copy link

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.
    I use dm binlog event filter to filer some binlog event.But i find that it is not working when configuration the uppercase event; Such as use “ALL DML” is not working. But us "all dml" is working

  2. What did you expect to see?
    Both use "ALL DML" and "all dml" are all working

  3. What did you see instead?
    I just use "all dml" instead "ALL DML"

  4. Versions of the cluster

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

Release Version: v1.0.0-alpha-70-g8bfa3e0
Git Commit Hash: 8bfa3e0
Git Branch: master
UTC Build Time: 2019-05-06 03:05:07
Go Version: go version go1.12 linux/amd64
```

- Upstream MySQL/MariaDB server version:

    ```
    5.7.20-log
    ```

- Downstream TiDB cluster version (execute `SELECT tidb_version();` in a MySQL client):

    ```

Release Version: v2.1.5-1-g0bd6b1b
Git Commit Hash: 0bd6b1b60816bf8a2022e657d7a472311548d82e
Git Branch: release-2.1
UTC Build Time: 2019-02-28 08:17:03
GoVersion: go version go1.11.2 linux/amd64
Race Enabled: false
TiKV Min Version: 2.1.0-alpha.1-ff3dd160846b7d1aed9079c389fc188f7f5ea13e
Check Table Before Drop: false
```

- How did you deploy DM: DM-Ansible or manually?

    ```
    DM-Ansible
    ```
@kalvinzhang kalvinzhang added the type/bug This issue is a bug report label Jun 21, 2019
@amyangfei
Copy link
Contributor

thanks for your report. Currently, DM supports lower case binlog event only and we are considering to make it case insensitive.

@amyangfei
Copy link
Contributor

we have supported case insensitive binlog event after this pr #188. Feel free to try the latest version of DM.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/bug This issue is a bug report
Projects
None yet
Development

No branches or pull requests

2 participants