-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
codec: support mysql Duration codec #64
Conversation
@@ -36,6 +38,7 @@ const ( | |||
formatFloatFlag = 'f' | |||
formatStringFlag = 's' | |||
formatBytesFlag = 'b' | |||
formatDuration = 't' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
formatDuration -> formatDurationFlag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, my fault.
b, r, err = DecodeInt(b) | ||
if err == nil { | ||
// use max fsp, let outer to do round manually. | ||
v[i] = mysql.Duration{Duration: time.Duration(r), Fsp: mysql.MaxFsp} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about encode fsp and decode it here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe no need here, but I am not sure.
As you know, we handle encode any fsp in table value codec or time marshal too, just let outer do the convert manually.
LGTM |
codec: support mysql Duration codec
Signed-off-by: Neil Shen <overvenus@gmail.com>
* disable remove scheduler and other pd related modifications when keyspace is set for BR Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com>
* BR adaption for keyspace feature (pingcap#38) Signed-off-by: iosmanthus <myosmanthustree@gmail.com> Co-authored-by: ystaticy <y_static_y@sina.com> Co-authored-by: disksing <i@disksing.com> * build br image (pingcap#62) * build br image Signed-off-by: disksing <i@disksing.com> * BR: Serverless Compatible BR (pingcap#64) * disable remove scheduler and other pd related modifications when keyspace is set for BR Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com> * make tidb br forward compatible (pingcap#84) * make tidb br forward compatible Signed-off-by: iosmanthus <myosmanthustree@gmail.com> * fix bug,when tables>0 and file=0 Signed-off-by: ystaticy <y_static_y@sina.com> * go mod tidy Signed-off-by: iosmanthus <myosmanthustree@gmail.com> Signed-off-by: iosmanthus <myosmanthustree@gmail.com> Signed-off-by: ystaticy <y_static_y@sina.com> Co-authored-by: ystaticy <y_static_y@sina.com> * Cse BR restore only download on leader (pingcap#120) * cse br restore,only download in region leader * support config Signed-off-by: ystaticy <y_static_y@sina.com> * cse br restore,only download in region leader Signed-off-by: ystaticy <y_static_y@sina.com> * cse br restore,only download in region leader Signed-off-by: ystaticy <y_static_y@sina.com> * fix logs Signed-off-by: ystaticy <y_static_y@sina.com> * fix logs Signed-off-by: ystaticy <y_static_y@sina.com> * fix logs Signed-off-by: ystaticy <y_static_y@sina.com> * remove unused code Signed-off-by: ystaticy <y_static_y@sina.com> * fix imports Signed-off-by: ystaticy <y_static_y@sina.com> * fix comments Signed-off-by: ystaticy <y_static_y@sina.com> * fix config of leaderdownload Signed-off-by: ystaticy <y_static_y@sina.com> * fix config of leaderdownload Signed-off-by: ystaticy <y_static_y@sina.com> * if met not leader error.return error immediately. Signed-off-by: ystaticy <y_static_y@sina.com> Signed-off-by: ystaticy <y_static_y@sina.com> * support skip split option for small dataset (pingcap#122) * support skip split option for small dataset Signed-off-by: iosmanthus <myosmanthustree@gmail.com> * disable SplitTable only when skip split Signed-off-by: iosmanthus <myosmanthustree@gmail.com> * rename api version of FileImporter Signed-off-by: iosmanthus <myosmanthustree@gmail.com> Signed-off-by: iosmanthus <myosmanthustree@gmail.com> Signed-off-by: iosmanthus <myosmanthustree@gmail.com> Signed-off-by: disksing <i@disksing.com> Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com> Signed-off-by: ystaticy <y_static_y@sina.com> Co-authored-by: ystaticy <y_static_y@sina.com> Co-authored-by: disksing <i@disksing.com> Co-authored-by: David <8039876+AmoebaProtozoa@users.noreply.github.com>
…ingcap#64) close pingcap#52049 Co-authored-by: 山岚 <36239017+YuJuncen@users.noreply.github.com>
No description provided.