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

Test failed on linux/386 #279

Closed
unknwon opened this issue Sep 26, 2015 · 17 comments
Closed

Test failed on linux/386 #279

unknwon opened this issue Sep 26, 2015 · 17 comments

Comments

@unknwon
Copy link
Contributor

unknwon commented Sep 26, 2015

I'm using Vagrant.

System: Linux squeeze-i386 2.6.32-5-686 #1 SMP Tue May 13 16:33:32 UTC 2014 i686 GNU/Linux

Error log:

godep go test -cover ./...
ok      github.com/pingcap/tidb 0.105s  coverage: 71.1% of statements
ok      github.com/pingcap/tidb/column  0.002s  coverage: 82.7% of statements
?       github.com/pingcap/tidb/context [no test files]
ok      github.com/pingcap/tidb/ddl 0.029s  coverage: 82.7% of statements
ok      github.com/pingcap/tidb/domain  0.005s  coverage: 86.7% of statements
ok      github.com/pingcap/tidb/expression  0.006s  coverage: 91.9% of statements
ok      github.com/pingcap/tidb/expression/builtin  0.003s  coverage: 95.7% of statements
ok      github.com/pingcap/tidb/expression/subquery 0.002s  coverage: 0.0% of statements
ok      github.com/pingcap/tidb/field   0.002s  coverage: 100.0% of statements
ok      github.com/pingcap/tidb/infoschema  0.005s  coverage: 98.6% of statements
?       github.com/pingcap/tidb/interpreter [no test files]
?       github.com/pingcap/tidb/kv  [no test files]
ok      github.com/pingcap/tidb/kv/memkv    1.443s  coverage: 81.8% of statements
ok      github.com/pingcap/tidb/meta    0.005s  coverage: 85.0% of statements
ok      github.com/pingcap/tidb/meta/autoid 0.003s  coverage: 89.5% of statements
ok      github.com/pingcap/tidb/model   0.002s  coverage: 100.0% of statements
ok      github.com/pingcap/tidb/mysqldef    0.009s  coverage: 89.6% of statements
ok      github.com/pingcap/tidb/parser  0.008s  coverage: 27.1% of statements
?       github.com/pingcap/tidb/parser/coldef   [no test files]
ok      github.com/pingcap/tidb/parser/opcode   0.001s  coverage: 100.0% of statements
?       github.com/pingcap/tidb/plan    [no test files]
ok      github.com/pingcap/tidb/plan/plans  0.043s  coverage: 76.5% of statements
?       github.com/pingcap/tidb/rset    [no test files]
ok      github.com/pingcap/tidb/rset/rsets  0.011s  coverage: 87.5% of statements
ok      github.com/pingcap/tidb/sessionctx  0.002s  coverage: 100.0% of statements
ok      github.com/pingcap/tidb/sessionctx/db   0.001s  coverage: 100.0% of statements
?       github.com/pingcap/tidb/sessionctx/forupdate    [no test files]
ok      github.com/pingcap/tidb/sessionctx/variable 0.002s  coverage: 37.1% of statements
?       github.com/pingcap/tidb/stmt    [no test files]
ok      github.com/pingcap/tidb/stmt/stmts  0.139s  coverage: 83.7% of statements
ok      github.com/pingcap/tidb/store/localstore    0.038s  coverage: 80.2% of statements
ok      github.com/pingcap/tidb/store/localstore/boltdb 0.014s  coverage: 86.4% of statements
?       github.com/pingcap/tidb/store/localstore/engine [no test files]
ok      github.com/pingcap/tidb/store/localstore/goleveldb  0.002s  coverage: 86.7% of statements
ok      github.com/pingcap/tidb/table   0.009s  coverage: 100.0% of statements
ok      github.com/pingcap/tidb/table/tables    0.025s  coverage: 75.2% of statements
?       github.com/pingcap/tidb/tidb-server [no test files]
ok      github.com/pingcap/tidb/tidb-server/server  0.132s  coverage: 64.0% of statements
ok      github.com/pingcap/tidb/util    0.004s  coverage: 73.1% of statements
ok      github.com/pingcap/tidb/util/arena  0.001s  coverage: 100.0% of statements
ok      github.com/pingcap/tidb/util/charset    0.001s  coverage: 96.6% of statements
ok      github.com/pingcap/tidb/util/codec  0.002s  coverage: 89.7% of statements
?       github.com/pingcap/tidb/util/errors [no test files]
ok      github.com/pingcap/tidb/util/errors2    0.001s  coverage: 100.0% of statements
ok      github.com/pingcap/tidb/util/format 0.001s  coverage: 81.2% of statements
ok      github.com/pingcap/tidb/util/hack   0.001s  coverage: 92.3% of statements
ok      github.com/pingcap/tidb/util/mock   0.001s  coverage: 100.0% of statements
?       github.com/pingcap/tidb/util/mock/mocks [no test files]
ok      github.com/pingcap/tidb/util/printer    0.001s  coverage: 89.8% of statements
2015/09/26 18:27:04 etc.go:370: [error] map[int]string

----------------------------------------------------------------------
FAIL: etc_test.go:244: testTypeEtcSuite.TestRoundFloat

etc_test.go:260:
    c.Assert(f, Equals, t.Expect)
... obtained float64 = -1
... expected float64 = -2

2015/09/26 18:27:04 etc.go:192: [error] unkown type 221, binary true
OOPS: 23 passed, 1 FAILED
--- FAIL: TestT (0.00s)
FAIL
coverage: 84.0% of statements
FAIL    github.com/pingcap/tidb/util/types  0.006s
godep: go exit status 1
make: *** [gotest] Error 1
@ngaut
Copy link
Member

ngaut commented Sep 27, 2015

@unknwon Does CI support i386 platform?

@unknwon
Copy link
Contributor Author

unknwon commented Sep 27, 2015

@ngaut
Copy link
Member

ngaut commented Sep 27, 2015

We don't have any i386 platform. And we are not gonna support i386 since we can't track the results if source changed. But PR is welcome :)
Could you help us to fix it ?

@unknwon
Copy link
Contributor Author

unknwon commented Nov 8, 2015

Update: same error occurs...

Maybe someday will be fixed automatically... :trollface:

@ngaut
Copy link
Member

ngaut commented Nov 9, 2015

oops, I guess we need CI for i386 platform.

@unknwon
Copy link
Contributor Author

unknwon commented Nov 9, 2015

It looks like a small failed test case, but I don't know the details, so I can't really tell... 😓

@qiuyesuifeng
Copy link
Member

@unknwon
Can u use this branch
https://github.com/pingcap/tidb/tree/qiuyesuifeng/issue-279
to have a try?
I guess we should use math.Abs for check when the float64 value is negative in RoundFloat function.

Thanks.

@siddontang
Copy link
Member

@unknwon

http://play.golang.org/p/FxT4zzRyQS

or you can try the RoundFloat instead and check again. :-)

@unknwon
Copy link
Contributor Author

unknwon commented Nov 9, 2015

I need to boot up my vagrant and use VIM 😵

I'll report back ASAP.

@unknwon
Copy link
Contributor Author

unknwon commented Nov 9, 2015

@qiuyesuifeng passed for that branch :trollface:

@qiuyesuifeng
Copy link
Member

@unknwon
Thanks. How about @siddontang function implement? Seems much better:-)

@unknwon
Copy link
Contributor Author

unknwon commented Nov 9, 2015

@qiuyesuifeng where should I modify the code? Same place as math.Abs?

@qiuyesuifeng qiuyesuifeng mentioned this issue Nov 9, 2015
@qiuyesuifeng
Copy link
Member

I update branch and u can try again. Thanks:)

@unknwon
Copy link
Contributor Author

unknwon commented Nov 9, 2015

That's nice :D

But I just left the home, will be back about 12 hrs...

@unknwon
Copy link
Contributor Author

unknwon commented Nov 10, 2015

@qiuyesuifeng passed :D

@unknwon
Copy link
Contributor Author

unknwon commented Nov 11, 2015

Please close this issue after you merged into master branch.

@qiuyesuifeng
Copy link
Member

@unknwon Thanks.

YuJuncen pushed a commit to YuJuncen/tidb that referenced this issue Apr 23, 2021
* restore-util: Implement split/scatter (pingcap#274)

* implement split/scatter

Signed-off-by: 5kbpers <[email protected]>

* init test

Signed-off-by: 5kbpers <[email protected]>

* redesign output/input of the lib

Signed-off-by: 5kbpers <[email protected]>

* update dependency

Signed-off-by: 5kbpers <[email protected]>

* add commments and more tests

Signed-off-by: 5kbpers <[email protected]>

* add ScanRegions interface to Client

Signed-off-by: 5kbpers <[email protected]>

* fix potential data race

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* Apply suggestions from code review

Co-Authored-By: kennytm <[email protected]>

* Update pkg/restore-util/client.go

Co-Authored-By: kennytm <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* update dependency

Signed-off-by: 5kbpers <[email protected]>

* resolve conflicts

Signed-off-by: 5kbpers <[email protected]>

* fix prefix rewrite

Signed-off-by: 5kbpers <[email protected]>

* add RewriteRule/skip failed scatter region/retry the SplitRegion

Signed-off-by: 5kbpers <[email protected]>

* fix test

Signed-off-by: 5kbpers <[email protected]>

* check if region has peer

Signed-off-by: 5kbpers <[email protected]>

* more logs

Signed-off-by: 5kbpers <[email protected]>

* restore-util: add split retry interval (pingcap#277)

* reset dependencies to release-3.1

* add split retry interval

Signed-off-by: 5kbpers <[email protected]>

* fix go.sum

Signed-off-by: 5kbpers <[email protected]>

* restore-util: wait for scatter region sequentially  (pingcap#279)

* wait for scatter region sequentially

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* restore-util: add on split hook (pingcap#281)

* restore-util: add on split hook

Signed-off-by: Neil Shen <[email protected]>

* Nil check onSplit

Co-Authored-By: kennytm <[email protected]>

* restore-util: fix returned new region is nil (pingcap#283)

* restore-util: fix returned new region is nil

Signed-off-by: 5kbpers <[email protected]>

* more logs

Signed-off-by: 5kbpers <[email protected]>

* *: gofmt

Signed-off-by: 5kbpers <[email protected]>

* Apply suggestions from code review

Co-Authored-By: kennytm <[email protected]>

* fix log

Signed-off-by: 5kbpers <[email protected]>

* restore-util: call onSplit on splitByRewriteRules (pingcap#285)

Signed-off-by: Neil Shen <[email protected]>

* restore-util: fix overlapped error message (pingcap#293)

* restore-util: fix overlapped error message

Signed-off-by: 5kbpers <[email protected]>

* fix log message

Signed-off-by: 5kbpers <[email protected]>

* reduce error trace

Signed-off-by: 5kbpers <[email protected]>

* fix test

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* restore-util: log warning when cannot find matched rewrite rule (pingcap#299)

* restore-util: add method to set placement rules and store labels (pingcap#301)

* restore-util: add method to set placement rules and store labels

Signed-off-by: disksing <[email protected]>

* minor fix

Signed-off-by: disksing <[email protected]>

* address comment

Signed-off-by: disksing <[email protected]>

* add GetPlacementRules

Signed-off-by: disksing <[email protected]>

* fix test

Signed-off-by: disksing <[email protected]>

* restore-util: support batch split (pingcap#300)

* restore-util: support batch split

Signed-off-by: 5kbpers <[email protected]>

* go fmt

Signed-off-by: 5kbpers <[email protected]>

* Apply suggestions from code review

Co-Authored-By: kennytm <[email protected]>

* address commits

Signed-off-by: 5kbpers <[email protected]>

* Update pkg/restore-util/split.go

Co-Authored-By: kennytm <[email protected]>

* add onSplit callback

Signed-off-by: 5kbpers <[email protected]>

* fix test

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* restore-util: add upper bound time for waiting for scatter (pingcap#305)

* restore: fix scatter regions failed

Signed-off-by: 5kbpers <[email protected]>

* add log

Signed-off-by: 5kbpers <[email protected]>

* stop waiting for scatter after 3min

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* restore-util: fix wrong url (pingcap#306)

Signed-off-by: disksing <[email protected]>

* restore-util: add warning about unmatched table id (pingcap#313)

* restore-util: support table partition

Signed-off-by: 5kbpers <[email protected]>

* fix log

Signed-off-by: 5kbpers <[email protected]>

* warn table id does not match

Signed-off-by: 5kbpers <[email protected]>

* add unit tests

Signed-off-by: 5kbpers <[email protected]>

* Apply suggestions from code review

Co-Authored-By: Neil Shen <[email protected]>

* fix compile error

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* address comments

Signed-off-by: 5kbpers <[email protected]>

* fix test

Signed-off-by: 5kbpers <[email protected]>

Co-authored-by: Ian <[email protected]>
Co-authored-by: Neil Shen <[email protected]>

* *: prune tidb-tools

Signed-off-by: Neil Shen <[email protected]>

* restore: address linters suggestions

Signed-off-by: Neil Shen <[email protected]>

* restore: merge restoreutil into restore

Signed-off-by: Neil Shen <[email protected]>

* address comment

Signed-off-by: Neil Shen <[email protected]>

Co-authored-by: 5kbpers <[email protected]>
Co-authored-by: kennytm <[email protected]>
Co-authored-by: disksing <[email protected]>
Co-authored-by: Ian <[email protected]>
YuJuncen pushed a commit to YuJuncen/tidb that referenced this issue Apr 23, 2021
* send a batch of kv in encodeLoop

* regine test

* address comment

* Fix wrongly pass nil columnNames.

Should get column names after `parser.ReadRow()` that it's setted
while parser the row.

* Update lightning/restore/restore.go

Co-Authored-By: kennytm <[email protected]>

Co-authored-by: kennytm <[email protected]>
Co-authored-by: Jiahao Huang <[email protected]>
YuJuncen pushed a commit to YuJuncen/tidb that referenced this issue Apr 23, 2021
iosmanthus pushed a commit to iosmanthus/tidb that referenced this issue Feb 28, 2023
Signed-off-by: ystaticy <[email protected]>
rleungx pushed a commit to rleungx/tidb that referenced this issue Feb 26, 2024
* default resource group (pingcap#245)

* default resource group

Signed-off-by: disksing <[email protected]>

* enable resource control

Signed-off-by: disksing <[email protected]>

* add comments

Signed-off-by: disksing <[email protected]>

* move DefaultResourceGroup to config

Signed-off-by: disksing <[email protected]>

---------

Signed-off-by: disksing <[email protected]>
Co-authored-by: zzm <[email protected]>

* Resource Control: fix segmentation violation when keyspace not set (pingcap#251)

* fix segmentation violation when setting up ru

Signed-off-by: David <[email protected]>

* only setup resource control when cluster id not empty

Signed-off-by: David <[email protected]>

---------

Signed-off-by: David <[email protected]>

* skip ur (pingcap#279)

Signed-off-by: ystaticy <[email protected]>

* disable resource group sql interface (pingcap#309)

Signed-off-by: disksing <[email protected]>

* *: update pd client and increase maxWaitDuration (pingcap#321)

Signed-off-by: disksing <[email protected]>

* update pd client and ru config (pingcap#325)

Signed-off-by: disksing <[email protected]>

* update max wait duration (pingcap#351)

Signed-off-by: disksing <[email protected]>

* update the pd client and client go (pingcap#419)

Signed-off-by: nolouch <[email protected]>

* update main

Signed-off-by: disksing <[email protected]>

---------

Signed-off-by: disksing <[email protected]>
Signed-off-by: David <[email protected]>
Signed-off-by: ystaticy <[email protected]>
Signed-off-by: nolouch <[email protected]>
Co-authored-by: zzm <[email protected]>
Co-authored-by: David <[email protected]>
Co-authored-by: ystaticy <[email protected]>
Co-authored-by: ShuNing <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants