Performance problem with lift-json render 2.4 milestones #1156

Closed
mjanson opened this Issue Dec 9, 2011 · 4 comments

Comments

Projects
None yet
3 participants
@mjanson

mjanson commented Dec 9, 2011

I posted a question about this issue a couple of days ago on the Lift group to see if it was known--I did not receive any response so I am assuming it is not known:

In this example I am seeing a 12+ second render time
for a list of 50000 elements:

val decomposed = decompose(1 to 50000) 
log.debug("decomposed") 
val rendered = net.liftweb.json.render(decomposed) 
log.debug("rendered") 

resulting in:
14:16:42.401 [http-8080-2] DEBUG c.t.s.ServiceServlet - decomposed
14:16:55.147 [http-8080-2] DEBUG c.t.s.ServiceServlet - rendered
^^ 12+s elapsed

I have reproduced this in both M4 and M5 on a i7 quad and Snow Leopard--JDK is:

java -version
java version "1.6.0_29"
Java(TM) SE Runtime Environment (build 1.6.0_29-b11-402-10M3527)
Java HotSpot(TM) 64-Bit Server VM (build 20.4-b02-402, mixed mode)

@jonifreeman

This comment has been minimized.

Show comment
Hide comment
@jonifreeman

jonifreeman Dec 12, 2011

Member

(In [[r:0556f9fed6ced1621a6283a85a4098ec0cfde575]]) Simpler and faster way to punctuate lists (fixes #1156)

Branch: master

Member

jonifreeman commented Dec 12, 2011

(In [[r:0556f9fed6ced1621a6283a85a4098ec0cfde575]]) Simpler and faster way to punctuate lists (fixes #1156)

Branch: master

@jonifreeman

This comment has been minimized.

Show comment
Hide comment
@jonifreeman

jonifreeman Dec 12, 2011

Member

Not a proper fix after all. Reopening.

Member

jonifreeman commented Dec 12, 2011

Not a proper fix after all. Reopening.

@jonifreeman

This comment has been minimized.

Show comment
Hide comment
@jonifreeman

jonifreeman Dec 12, 2011

Member

Proper fix after all :)

Member

jonifreeman commented Dec 12, 2011

Proper fix after all :)

@github-importer

This comment has been minimized.

Show comment
Hide comment

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment