Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

logSink.units does not flush regularly #2424

Closed
dengqee opened this issue Jul 30, 2021 · 1 comment
Closed

logSink.units does not flush regularly #2424

dengqee opened this issue Jul 30, 2021 · 1 comment
Labels
area/ticdc Issues or PRs related to TiCDC. component/pitr Point-in-time recovery component. severity/major This is a major bug. type/bug This is a bug.

Comments

@dengqee
Copy link
Contributor

dengqee commented Jul 30, 2021

Bug Report

look 1259

https://github.com/pingcap/ticdc/blob/04e028419387871b80ddc95377751092f03f26ae/cdc/sink/cdclog/utils.go#L124-L139
There is a ticker to flush logSink.units every 500 ms, but why 1259 did not flush logSink.units?
So, I add a log, like this
image
I created a s3 sink changefeed:

cdc cli changefeed create --sink-uri="s3://tidb-test/cdclog/?endpoint=http://xxxx:9000&access-Key=xxxx&secret-Access-Key=xxxx"  --changefeed-id=s3-sink --start-ts 426674072187043841 --pd http://xxxx:2379

insert data by sysbench

sysbench --mysql-host=xxxx --mysql-user=root --mysql-port=4000 --mysql-db=test oltp_insert --tables=1 --mysql-password='xxxx' --table-size=200000 prepare

I got this:
image

https://github.com/pingcap/ticdc/blob/04e028419387871b80ddc95377751092f03f26ae/cdc/sink/cdclog/s3.go#L80-L82
unit.size < 5MB, so shouldFlush return false, but utils.dataCh chennal is full, so changefeed is bloked.

Please answer these questions before submitting your issue. Thanks!

  1. Versions of the cluster

    • Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):

       v4.0.10
      
    • TiCDC version (execute cdc version):

      v4.0.10
      
@amyangfei
Copy link
Contributor

closed by #2431. Thanks for your contribution

Question and Bug Reports automation moved this from Need Triage to Done Aug 3, 2021
@AkiraXie AkiraXie added the area/ticdc Issues or PRs related to TiCDC. label Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ticdc Issues or PRs related to TiCDC. component/pitr Point-in-time recovery component. severity/major This is a major bug. type/bug This is a bug.
Development

No branches or pull requests

3 participants