Skip to content

Commit

Permalink
config(ticdc): change case-sensitive default value (#10049)
Browse files Browse the repository at this point in the history
close #10047
  • Loading branch information
sdojjy authored Nov 10, 2023
1 parent b2011a6 commit db3a2dd
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion cdc/api/v2/model_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ import (
// note: this is api published default value, not change it
var defaultAPIConfig = &ReplicaConfig{
MemoryQuota: config.DefaultChangefeedMemoryQuota,
CaseSensitive: true,
CaseSensitive: false,
CheckGCSafePoint: true,
BDRMode: util.AddressOf(false),
EnableSyncPoint: util.AddressOf(false),
Expand Down
2 changes: 1 addition & 1 deletion cdc/model/changefeed_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ func TestVerifyAndComplete(t *testing.T) {
StartTs: 417257993615179777,
Config: &config.ReplicaConfig{
MemoryQuota: 1073741824,
CaseSensitive: true,
CaseSensitive: false,
CheckGCSafePoint: true,
SyncPointInterval: util.AddressOf(time.Minute * 10),
SyncPointRetention: util.AddressOf(time.Hour * 24),
Expand Down
2 changes: 1 addition & 1 deletion pkg/config/replica_config.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ const (

var defaultReplicaConfig = &ReplicaConfig{
MemoryQuota: DefaultChangefeedMemoryQuota,
CaseSensitive: true,
CaseSensitive: false,
CheckGCSafePoint: true,
EnableSyncPoint: util.AddressOf(false),
SyncPointInterval: util.AddressOf(10 * time.Minute),
Expand Down
2 changes: 1 addition & 1 deletion tests/integration_tests/api_v2/cases.go
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ var customReplicaConfig = &ReplicaConfig{
// defaultReplicaConfig check if the default values is changed
var defaultReplicaConfig = &ReplicaConfig{
MemoryQuota: 1024 * 1024 * 1024,
CaseSensitive: true,
CaseSensitive: false,
CheckGCSafePoint: true,
Filter: &FilterConfig{
Rules: []string{"*.*"},
Expand Down
4 changes: 2 additions & 2 deletions tests/integration_tests/cli/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ function run() {

# Update changefeed failed because changefeed is running
cat - >"$WORK_DIR/changefeed.toml" <<EOF
case-sensitive = false
case-sensitive = true
[scheduler]
enable-table-across-nodes = true
EOF
Expand All @@ -91,7 +91,7 @@ EOF
# Update changefeed
run_cdc_cli changefeed update --pd=$pd_addr --config="$WORK_DIR/changefeed.toml" --no-confirm --changefeed-id $uuid
changefeed_info=$(curl -s -X GET "http://127.0.0.1:8300/api/v2/changefeeds/$uuid/meta_info" 2>&1)
if [[ ! $changefeed_info == *"\"case_sensitive\":false"* ]]; then
if [[ ! $changefeed_info == *"\"case_sensitive\":true"* ]]; then
echo "[$(date)] <<<<< changefeed info is not updated as expected ${changefeed_info} >>>>>"
exit 1
fi
Expand Down

0 comments on commit db3a2dd

Please sign in to comment.