{"action":"closed","issue":{"url":"https://api.github.com/repos/ReactiveX/RxJava/issues/3368","labels_url":"https://api.github.com/repos/ReactiveX/RxJava/issues/3368/labels{/name}","comments_url":"https://api.github.com/repos/ReactiveX/RxJava/issues/3368/comments","events_url":"https://api.github.com/repos/ReactiveX/RxJava/issues/3368/events","html_url":"https://github.com/ReactiveX/RxJava/issues/3368","id":107463179,"number":3368,"title":"Hundreds of 2.x operators/classes for NbpObservable","user":{"login":"akarnokd","id":1269832,"avatar_url":"https://avatars.githubusercontent.com/u/1269832?v=3","gravatar_id":"","url":"https://api.github.com/users/akarnokd","html_url":"https://github.com/akarnokd","followers_url":"https://api.github.com/users/akarnokd/followers","following_url":"https://api.github.com/users/akarnokd/following{/other_user}","gists_url":"https://api.github.com/users/akarnokd/gists{/gist_id}","starred_url":"https://api.github.com/users/akarnokd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akarnokd/subscriptions","organizations_url":"https://api.github.com/users/akarnokd/orgs","repos_url":"https://api.github.com/users/akarnokd/repos","events_url":"https://api.github.com/users/akarnokd/events{/privacy}","received_events_url":"https://api.github.com/users/akarnokd/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ReactiveX/RxJava/labels/Question","name":"Question","color":"cc317c"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/ReactiveX/RxJava/milestones/9","html_url":"https://github.com/ReactiveX/RxJava/milestones/2.0","labels_url":"https://api.github.com/repos/ReactiveX/RxJava/milestones/9/labels","id":755421,"number":9,"title":"2.0","description":"Major new version of RxJava.\r\n\r\nIt may be in a new package (io.reactivex) so it can live alongside RxJava 1.x (rx.*).\r\n\r\nNOTE: There is no guarantee this will happen for Java 8. Version 2 may wait until Java 9.","creator":{"login":"benjchristensen","id":813492,"avatar_url":"https://avatars.githubusercontent.com/u/813492?v=3","gravatar_id":"","url":"https://api.github.com/users/benjchristensen","html_url":"https://github.com/benjchristensen","followers_url":"https://api.github.com/users/benjchristensen/followers","following_url":"https://api.github.com/users/benjchristensen/following{/other_user}","gists_url":"https://api.github.com/users/benjchristensen/gists{/gist_id}","starred_url":"https://api.github.com/users/benjchristensen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjchristensen/subscriptions","organizations_url":"https://api.github.com/users/benjchristensen/orgs","repos_url":"https://api.github.com/users/benjchristensen/repos","events_url":"https://api.github.com/users/benjchristensen/events{/privacy}","received_events_url":"https://api.github.com/users/benjchristensen/received_events","type":"User","site_admin":false},"open_issues":29,"closed_issues":141,"state":"open","created_at":"2014-08-15T19:30:54Z","updated_at":"2015-09-25T18:15:51Z","due_on":null,"closed_at":null},"comments":3,"created_at":"2015-09-21T08:14:33Z","updated_at":"2015-09-25T18:15:51Z","closed_at":"2015-09-25T18:15:51Z","body":"I believe I can derive non-backpressure operators from backpressured operators with a relatively simple process (delete backpressure related fields, fix up the classes). Such task would take me 1-1.5 days tops and I can do it in a single PR. The work, unlike the run on the backpressured, can now be done in parallel as there aren't really and cross-dependencies or algorithmic challenges; it's mostly a simple mechanical process. Once all of them is \"ported\", the NbpObservable can be modified to access them in one PR.\r\n\r\nHowever, this is at least 20kloc for classes and another 20kloc for unit test where such large amount of change is practically non-reviewable.\r\n\r\nMy question is, how should this be handled? One by one and with 200+ PRs or batched into ~1kloc chunks and thus within ~40 PRs?"}}