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
slow compilation on chained calls to polymorphic method with type alias in siganature (akka-streams Flow.map) #9157
Comments
Imported From: https://issues.scala-lang.org/browse/SI-9157?orig=1 |
@rkuhn said: |
@retronym said: Here's a smaller repro: trait Flow[-In, +Out] {
type Repr[+O] <: Flow[In, O]
def map: Repr[String]
}
class Test {
def slowFlow(f: Flow[String,String]#Repr[String]#Repr[String]#Repr[String]#Repr[String]#Repr[String]#Repr[String]#Repr[String]#Repr[String]#Repr[String]) = {
f.map
}
} As @odersky noted on the mailing list, this typechecks promptly in dotty. We've must have some exponential performance bug in scalac. |
@retronym said (edited on Feb 18, 2015 2:34:37 AM UTC):
|
Alexey Romanchuk (13h3r) said: |
@retronym said: |
@adriaanm said: |
@rkuhn said: |
@paulp said: The preference of just-reported bugs over long neglected bugs creates all the wrong incentives. I should probably close my 142 open tickets and re-open them one per day over the next five months. |
@adriaanm said: What are these wrong incentives you speak of? Not to look for existing bugs when reporting a new one? We're just going to have to accept we won't be able to fix all the open bugs with the resources we have, let alone keep all the tickets in our heads to spot duplicates as readily as you do. In the end, this is like a "jira chapter 11", a soft version of the bankruptcy idea that was floated on twitter not too long ago, except that declaring such "defeat" explicitly (which we have discussed internally multiple times) would probably be met with "ha! they imploded!" cheers if it came from us. I do understand your frustration, and wish we could address it, but I really don't see the point of all this negativity given that we are already doing the best we can. |
@paulp said: |
@paulp said: |
@adriaanm said (edited on Feb 19, 2015 6:40:33 PM UTC): |
@paulp said:
So 27. Okay, with a variation on that I get the coarse look:
And I can never remember how to accomplish multi-line stuff via sed and friends so I just manually deleted it down to the ones without comments. [SI-2936] [SI-3459] [SI-5172] [SI-5505] [SI-6533] [SI-6598] [SI-6717] Must have one straggler in there since that's 28. |
@paulp said: |
@paulp said: |
@retronym said (edited on Feb 19, 2015 10:18:19 PM UTC): Always happy to kill N birds with M stones where N > M, but I agree it would be better to avoid duplicate birds from taking flight in the first place. I do close a lot of new tickets as duplicates, but don't find the old ones, and sometimes forget to look. Thankfully in this case the minimization and fix were both surprisingly forthcoming. I'll take a look through that list, too. |
@paulp said: In addition to the 28 entirely uncommented tickets noted above, these are "effectively uncommented" as their only comments are from me, from various third parties, and/or from Joe Jirabot with "Unassigning and rescheduling to M6 as previous deadline was missed" or some variation thereof. #3197 #4323 #4381 #4401 #4663 #4721 #4739 #4867 Here are a couple where I feel the conversation was left unreasonably hanging. Seeing all 140 of those tickets one after another, I had a glimpse of what it must have been like to be a civil war battlefield medic. Fighting for the South. |
As reported https://groups.google.com/d/msg/scala-internals/wdQVas4jNdc/vzLp2uOdITsJ
The text was updated successfully, but these errors were encountered: