ddl: fix corrupted default value for bit type column (#18036)

[ti-srebot]

cherry-pick #18036 to release-4.0

---

What problem does this PR solve?

Issue Number: close #17641 close #17642

Problem Summary:

The ColumnInfo.OriginDefaultValue is corrupted when the column type is BIT. BIT type column values are stored as strings by JSON marshaling and read it by unmarshaling later. However, the data in a string can be changed in this case:

originBytes := []byte{25, 185}
originStr := string(originBytes)

mr, _ := json.Marshal(originStr)
var jsoned string
_ = json.Unmarshal(mr, &jsoned)

fmt.Printf("%v\n", []byte(originStr))
fmt.Printf("%v\n", []byte(jsoned))

[25 185]
[25 239 191 189]

If we store it as []byte, the problem disappeared. This is how DefaultValueBit works.

type ColumnInfo struct {
	// ...
	OriginDefaultValue    interface{} `json:"origin_default"`
	DefaultValue          interface{} `json:"default"`
	DefaultValueBit       []byte      `json:"default_bit"`
	// ...
}

The same problem exists in OriginDefaultValue. PR #12168 tried to restore the OriginDefaultValue from DefaultValueBit in the getter, but OriginDefaultValue is different from DefaultValue: OriginDefaultValue is used to skip the backfilling phase of AddColumn(). If a column's default value is modified by ALTER TABLE MODIFY COLUMN DEFAULT, they are not equal anymore(see the example in #17641).

What is changed and how it works?

This PR adds a field OriginDefaultValueBit to ColumnInfo, which borrows from DefaultValueBit to solve the problem completely.

Proposal: xxx

What's Changed:

• Getter and setter for OriginDefaultValue.
• A simple test case in Incorrect default value for bit type column #17641.

How it Works:
If the column type is BIT, set/get the content with OriginDefaultValueBit.

Related changes

• Need to cherry-pick to the release branch

Check List

Tests

• Unit test
• Integration test

Side effects

Release note

• fix corrupted default value for bit type column.

[ti-srebot]

/run-all-tests

[tangenta]

/run-all-tests

[tangenta]

/run-all-tests

[tangenta]

Test Failed Log:

[2020-10-22T09:07:01.872Z] java.net.SocketTimeoutException: sent ping but didn't receive pong within 30000ms (after 4 successful ping/pongs)
[2020-10-22T09:07:01.872Z] 	at okhttp3.internal.ws.RealWebSocket.writePingFrame(RealWebSocket.java:546)
[2020-10-22T09:07:01.872Z] 	at okhttp3.internal.ws.RealWebSocket$PingRunnable.run(RealWebSocket.java:530)
[2020-10-22T09:07:01.872Z] 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
[2020-10-22T09:07:01.872Z] 	at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308)
[2020-10-22T09:07:01.873Z] 	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180)
[2020-10-22T09:07:01.873Z] 	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294)
[2020-10-22T09:07:01.873Z] 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[2020-10-22T09:07:01.873Z] 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[2020-10-22T09:07:01.873Z] 	at java.lang.Thread.run(Thread.java:748)
[2020-10-22T09:07:08.759Z] Cannot contact test-go1130-memvolume-m5m30: hudson.remoting.RequestAbortedException: java.nio.channels.ClosedChannelException
[2020-10-22T09:14:01.683Z] Could not connect to test-go1130-memvolume-m5m30 to send interrupt signal to process

[tangenta]

/run-all-tests

[AilinKid]

LGTM

[wjhuang2016]

LGTM

[bb7133]

/run-sqllogic-test
/run-unit-test