[JIRA] (JENKINS-42221) MSBuild parser takes unreasonable amount of time

2017-02-21 Thread emil.sty...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Emil Styrke created an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-42221  
 
 
  MSBuild parser takes unreasonable amount of time   
 

  
 
 
 
 

 
Issue Type: 
  Bug  
 
 
Assignee: 
 Ulli Hafner  
 
 
Components: 
 warnings-plugin  
 
 
Created: 
 2017/Feb/21 3:26 PM  
 
 
Environment: 
 warnings plugin 4.59  
 
 
Priority: 
  Major  
 
 
Reporter: 
 Emil Styrke  
 

  
 
 
 
 

 
 Parsing a log with the MSBuild parser takes over 10 minutes in severe cases, whereas the GCC parser takes at most 10 seconds on a similar log. I'll admit that the log is rather large (17 Mb), but the performance between GCC and MSBuild differs by a factor of 60. This log is the result of a Pipeline build which makes 14 parallel sub-builds on different compilers and platforms, and one possibility to lessen the severity of this issue would be to be able to run the parser on just the current "node" or "stage" statement in the Pipeline script. This would prevent the MSBuild parser from having to see the gazillion of lines coming from GCC, which it won't recognize anyway.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  

[JIRA] (JENKINS-42221) MSBuild parser takes unreasonable amount of time

2017-02-21 Thread ullrich.haf...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Ulli Hafner commented on  JENKINS-42221  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: MSBuild parser takes unreasonable amount of time   
 

  
 
 
 
 

 
 After fixing several issues the MS build parser regular _expression_ seems to be overly complex now. I think this is the problem. You can try to pipe the output of the compile steps into files and use the parser only on the corresponding files too speed up things...   
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-42221) MSBuild parser takes unreasonable amount of time

2017-02-22 Thread emil.sty...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Emil Styrke commented on  JENKINS-42221  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: MSBuild parser takes unreasonable amount of time   
 

  
 
 
 
 

 
 Thanks for the reply! I did some benchmarking on the regex, and I found out that adding a beginning-of-line anchor to the first "or" group cuts the running time from 16 seconds to 0.15 seconds in my testcase. I created a PR for the change at https://github.com/jenkinsci/warnings-plugin/pull/86.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-42221) MSBuild parser takes unreasonable amount of time

2017-02-22 Thread scm_issue_l...@java.net (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 SCM/JIRA link daemon commented on  JENKINS-42221  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: MSBuild parser takes unreasonable amount of time   
 

  
 
 
 
 

 
 Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/warnings/parser/MsBuildParser.java http://jenkins-ci.org/commit/warnings-plugin/ad9f3c917876abbb08763a6b041934a6e6b404f4 Log: [FIXED JENKINS-42221] Merge pull request #86 from estyrke/patch-1 Added anchor to improve performance of regular _expression_ of MS Build parser. Compare: https://github.com/jenkinsci/warnings-plugin/compare/b49d2b85a32f...ad9f3c917876  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-42221) MSBuild parser takes unreasonable amount of time

2017-02-22 Thread scm_issue_l...@java.net (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 SCM/JIRA link daemon resolved as Fixed  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-42221  
 
 
  MSBuild parser takes unreasonable amount of time   
 

  
 
 
 
 

 
Change By: 
 SCM/JIRA link daemon  
 
 
Status: 
 Open Resolved  
 
 
Resolution: 
 Fixed  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-42221) MSBuild parser takes unreasonable amount of time

2017-02-22 Thread scm_issue_l...@java.net (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 SCM/JIRA link daemon commented on  JENKINS-42221  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: MSBuild parser takes unreasonable amount of time   
 

  
 
 
 
 

 
 Code changed in jenkins User: Emil Styrke Path: src/main/java/hudson/plugins/warnings/parser/MsBuildParser.java http://jenkins-ci.org/commit/warnings-plugin/3bd036172840f914b75ff831c85a8f126225dc0d Log: Add anchor to improve performance Should fix JENKINS-42221.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.