Using thread pool to transform records#9
Using thread pool to transform records#9allenwq wants to merge 2 commits intolowjoel:masterfrom allenwq:multi-threading
Conversation
|
@lowjoel No idea why I get |
lib/database_transform/schema.rb
Outdated
| delegate :around_job, :thread, to: :class | ||
| class << self | ||
| # Store the block if a block is given, otherwise the stored block will be returned. | ||
| def around_job(&block) |
There was a problem hiding this comment.
I would define these outside the self singleton class, to be consistent with transform_table
There was a problem hiding this comment.
also, should around_job be composable? Multiple around_job should cause it to chain?
There was a problem hiding this comment.
I can't think of a use case for multiple around jobs ><
There was a problem hiding this comment.
yes, but that's the behaviour expected of most Ruby around_X callbacks.
|
@lowjoel I tried included a Gemfile.lock (make sure travis uses the same gems as my local) and change the ruby versions, all does not work very well... |
|
@allenwq gemfile.lock doesn't affect gems, so that won't work. What are you trying to achieve? |
|
It affects the gems install on travis for testing right ? |
|
Yes, but what's the problem you're trying to solve? |
|
Oh, Tests passed on my local, but cannot even run on travis |
|
I don't think it's a gem version problem. Have you tried with a linux VM? |
|
...master fails on my linux server > < |
|
So it's not my PR breaks the travis, @lowjoel can you verify ? |
|
@allenwq https://travis-ci.org/lowjoel/database_transform/builds/126300182, just committed 2 hours ago, finished running 1 minute ago... |
|
Sorry, I was using the wrong master > < But now I can get it fail locally, will debug and get back to you. |
|
Thanks! I wanted to try but my VirtualBox VM can't get internet... |
|
Wait.. even I changed the master it's still failing, I was in commit a913457 ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-linux] It's weird that your commit passes travis and it fails on my linux with the same exception as my PR. |
Pending to add README..