Remove ErrGCTTLExceeded changefeed panic

[3AceShowHand]

What did you do?

tiup playground v.5.1.1

update mysql.tidb set VARIABLE_VALUE="10s" where VARIABLE_NAME="tikv_gc_run_interval";

./bin/cdc server --pd=http://127.0.0.1:2379 --gc-ttl=120

./bin/cdc cli changefeed create --pd=http://127.0.0.1:2379 --sink-uri="mysql://root:123456@127.0.0.1:3306/" --changefeed-id="gcttl-test"

./bin/cdc cli changefeed pause --pd=http://127.0.0.1:2379 --changefeed-id="gcttl-test"

wait for a few minutes, changefeed's status become error, then:

./bin/cdc cli changefeed remove --pd=http://127.0.0.1:2379 --changefeed-id="gcttl-test"

What did you expect to see?

changefeed removed successfully.

What did you see instead?

Versions of the cluster

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

v5.1.1

TiCDC version (execute cdc version):

v5.1.1

ticdc.log

[zhaoxinyu]

This bug cannot be reproduced on v5.3.0. The immediate cause for this bug is passing a nil value to s.Func. See

https://github.com/pingcap/tiflow/blob/v5.1.1/pkg/orchestrator/interfaces.go#L55

[3AceShowHand]

it's fine to close the issue.

[zhaoxinyu]

it's fine to close the issue.

The nil pointer still needs to be checked. I will submit a PR later.