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 quora: reduce checking #190

Merged
merged 2 commits into from
Jan 20, 2024

Conversation

tikkss
Copy link
Contributor

@tikkss tikkss commented Jan 19, 2024

Because tsv file is too big(404,290 rows).

Before this change:

$ time ruby -I lib test/run-test.rb -t "/QuoraDuplicateQuestionPairTest/" -v
Loaded suite test
Started
QuoraDuplicateQuestionPairTest:
  test: #each:                                                         .: (17.476536)

Finished in 17.476912 seconds.
-------------------------------------------------------------------------------------
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
-------------------------------------------------------------------------------------
0.06 tests/s, 0.06 assertions/s

real    0m18.117s
user    0m17.554s
sys     0m0.383s

After this change:

$ time ruby -I lib test/run-test.rb -t "/QuoraDuplicateQuestionPairTest/" -v
Loaded suite test
Started
QuoraDuplicateQuestionPairTest:
  test: #each:                                                          .: (0.001124)

Finished in 0.001461 seconds.
-------------------------------------------------------------------------------------
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
-------------------------------------------------------------------------------------
684.46 tests/s, 684.46 assertions/s

real    0m0.595s
user    0m0.376s
sys     0m0.126s

Refs #188.

Because tsv file is too big(404,290 rows).

Before this change:

```bash
$ time ruby -I lib test/run-test.rb -t "/QuoraDuplicateQuestionPairTest/" -v
Loaded suite test
Started
QuoraDuplicateQuestionPairTest:
  test: #each:                                                         .: (17.476536)

Finished in 17.476912 seconds.
-------------------------------------------------------------------------------------
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
-------------------------------------------------------------------------------------
0.06 tests/s, 0.06 assertions/s

real    0m18.117s
user    0m17.554s
sys     0m0.383s
```

After this change:

```bash
$ time ruby -I lib test/run-test.rb -t "/QuoraDuplicateQuestionPairTest/" -v
Loaded suite test
Started
QuoraDuplicateQuestionPairTest:
  test: #each:                                                          .: (0.001124)

Finished in 0.001461 seconds.
-------------------------------------------------------------------------------------
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
-------------------------------------------------------------------------------------
684.46 tests/s, 684.46 assertions/s

real    0m0.595s
user    0m0.376s
sys     0m0.126s
```

Refs red-data-tools#188.
Because this is a test for #each

Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
@kou kou merged commit 37e2f35 into red-data-tools:master Jan 20, 2024
12 checks passed
@kou
Copy link
Member

kou commented Jan 20, 2024

Thanks!

@tikkss
Copy link
Contributor Author

tikkss commented Jan 20, 2024

Thanks for your review and suggestion!

@tikkss tikkss deleted the test-quora-reduce-checking branch January 21, 2024 00:11
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

Successfully merging this pull request may close these issues.

2 participants