Re: Calcite - RelMetadataTest.testPullUpPredicatesForExprsItr - 20 second timeout
Yeah, I've also taken a look but it doesn't seemed connected - but after a while I've started suspecting that my I had too many back noise...
I instead went the other way; and looked at it with a profiler; and it seems to me that there are parts of an old 2-3 year old logic; which seems to be redundant with the current rexsimplify workings....I'm still confirming this..sorry for writing just now...I wanted to finish with investigation first...
On 26 June 2018 19:53:57 CEST, Michael Mior <mmior@xxxxxxxxxx> wrote:
>I had some time to dig into this today. It looks like it was this
>which started slowing things down
>I'm not too familiar with this part of the code, but if anyone has
>suggestions on improving, that would be great :)
>Le jeu. 21 juin 2018 à 19:43, Julian Hyde <jhyde@xxxxxxxxxx> a écrit :
>> I noticed this test slow down about a month ago. I hit the timeouts
>> assumed at the time that I was running on an underpowered/overtaxed
>> Can you log a JIRA case please, Kevin?
>> It would be useful if someone could do some experiments to find out
>> whether this test was faster 6 months ago, and if so, which git
>> since then caused the slowdown.
>> > On Jun 21, 2018, at 12:24 PM, Kevin Risden <krisden@xxxxxxxxxx>
>> > I have been hitting the
>> > second timeout more often recently and Apache build Jenkins has hit
>> > recently as well [1,2,3]. In one of the recent partial successes
>> > RelMetadataTest suite takes just under 20 seconds at 19.45 seconds.
>> > [INFO] Running org.apache.calcite.test.RelMetadataTest
>> > [WARNING] Tests run: 140, Failures: 0, Errors: 0, Skipped: 6, Time
>> > elapsed: 19.45 s - in org.apache.calcite.test.RelMetadataTest
>> > I don't know much about what this test is trying to do. Is there
>> > that could be causing this test to run longer?
>> >  https://builds.apache.org/job/Calcite-Master/306/
>> >  https://builds.apache.org/job/Calcite-Master/307/
>> >  https://builds.apache.org/job/Calcite-Master/308/
>> > 
>> > Kevin Risden