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

*: fix bad column offsets. #3754

Merged
merged 37 commits into from
Jul 28, 2017
Merged
Changes from 1 commit
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
669e2e8
*: fix bad column offsets.
hicqu Jul 14, 2017
50c258c
fix ci.
hicqu Jul 14, 2017
631a8ca
improve code.
hicqu Jul 14, 2017
c7a74c7
fix bug.
hicqu Jul 14, 2017
d2f8a91
Merge branch 'master' into qupeng/column-offset
hicqu Jul 14, 2017
b297477
Merge branch 'master' into qupeng/column-offset
hicqu Jul 17, 2017
22bb700
Merge branch 'master' into qupeng/column-offset
hicqu Jul 17, 2017
93240dd
address comments.
hicqu Jul 18, 2017
c8dde90
Merge branch 'master' into qupeng/column-offset
hicqu Jul 18, 2017
96a667a
fix ci.
hicqu Jul 18, 2017
397a44d
Merge branch 'master' into qupeng/column-offset
hicqu Jul 18, 2017
5b1cddc
address comments.
hicqu Jul 19, 2017
5846f7f
Merge branch 'master' into qupeng/column-offset
hicqu Jul 19, 2017
e20cb89
Merge branch 'master' into qupeng/column-offset
hicqu Jul 20, 2017
d65a7f3
address comments.
hicqu Jul 20, 2017
d8fb37b
Merge branch 'master' into qupeng/column-offset
hicqu Jul 20, 2017
f205635
address comments.
hicqu Jul 20, 2017
3d88b0f
Merge branch 'master' into qupeng/column-offset
hicqu Jul 20, 2017
d76b1e3
Merge branch 'qupeng/column-offset' of github.com:pingcap/tidb into q…
hicqu Jul 20, 2017
9d2b86a
Merge branch 'master' into qupeng/column-offset
hicqu Jul 21, 2017
42b202c
address comments.
hicqu Jul 21, 2017
ac2aa3c
address comments.
hicqu Jul 21, 2017
7383b86
Merge branch 'master' into qupeng/column-offset
hicqu Jul 21, 2017
5b0883d
address comments.
hicqu Jul 21, 2017
cbba317
address comments.
hicqu Jul 21, 2017
bb70730
Merge branch 'master' into qupeng/column-offset
hicqu Jul 21, 2017
906cf4e
fix bad comment.
hicqu Jul 21, 2017
370c1e4
Merge branch 'master' into qupeng/column-offset
hicqu Jul 21, 2017
1269693
Merge branch 'master' into qupeng/column-offset
hicqu Jul 24, 2017
f8f0df2
Merge branch 'master' into qupeng/column-offset
hicqu Jul 25, 2017
5b9e9e5
address comments.
hicqu Jul 25, 2017
7665b13
Merge branch 'master' into qupeng/column-offset
hicqu Jul 25, 2017
2f7b85f
Merge branch 'master' into qupeng/column-offset
hicqu Jul 26, 2017
1661ec3
ddl: fix ddl_db_change_test.go.
hicqu Jul 26, 2017
42939df
address comments.
hicqu Jul 27, 2017
dbf52ea
Merge branch 'master' into qupeng/column-offset
zimulala Jul 28, 2017
044478d
Merge branch 'master' into qupeng/column-offset
hicqu Jul 28, 2017
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix ci.
  • Loading branch information
hicqu committed Jul 18, 2017
commit 96a667ae56a2c7c4cddf441d289ade85d75a39b1
2 changes: 1 addition & 1 deletion executor/write.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ var (
func updateRecord(ctx context.Context, h int64, oldData, newData []types.Datum, touched []bool, t table.Table, onDup bool) (bool, error) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add comment for touched parameter.

var sc = ctx.GetSessionVars().StmtCtx
var changed, handleChanged = false, false
var onUpdateNoChange map[int]bool
var onUpdateNoChange = make(map[int]bool)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add comment for this variable.


// We can iterate on public columns not writable columns,
// because all of them are sorted by their `Offset`, which
Expand Down