1. #11
    Sencha Premium Member
    Join Date
    Sep 2009
    Posts
    42
    Vote Rating
    1
    op1op is on a distinguished road

      0  

    Default


    4.x grid is too bad.
    why load record slower than 3.4?
    4.x load cost more than 1 second,3.4 cost less then 0.2 second.

  2. #12
    Ext Premium Member halcwb's Avatar
    Join Date
    Mar 2010
    Location
    Rotterdam
    Posts
    385
    Vote Rating
    54
    halcwb is a jewel in the rough halcwb is a jewel in the rough halcwb is a jewel in the rough

      0  

    Default


    As a suggestion, you might wrap the load between a grid.suspendEvents and grid.resumeEvents, this way the grid won't trigger any reloading in between the initial load.

  3. #13
    Ext JS Premium Member
    Join Date
    Jun 2011
    Location
    St. Louis
    Posts
    212
    Vote Rating
    9
    jimmylu98 will become famous soon enough

      0  

    Default


    Quote Originally Posted by halcwb View Post
    As a suggestion, you might wrap the load between a grid.suspendEvents and grid.resumeEvents, this way the grid won't trigger any reloading in between the initial load.
    Are you talking about the code like below? If yes, it is still need 3 layouts same as without grid.suspendEvents and grid.resumeEvents and the performance is about same.

    PHP Code:
                render: {
                    
    fn: function(){
                var 
    me this;
                   
    me.suspendEvents(); 
                            
    store.load();
                   
    me.resumeEvents();
                    }
                } 
    As I said, the patch in this thread will reduce 3 layouts to 2 layouts( same as extjs 3.4.x), it also prevents any layout due to column show/hide and grouping.

  4. #14
    Ext Premium Member halcwb's Avatar
    Join Date
    Mar 2010
    Location
    Rotterdam
    Posts
    385
    Vote Rating
    54
    halcwb is a jewel in the rough halcwb is a jewel in the rough halcwb is a jewel in the rough

      0  

    Default


    Quote Originally Posted by jimmylu98 View Post
    Are you talking about the code like below? If yes, it is still need 3 layouts same as without grid.suspendEvents and grid.resumeEvents and the performance is about same.

    PHP Code:
                render: {
                    
    fn: function(){
                var 
    me this;
                   
    me.suspendEvents(); 
                            
    store.load();
                   
    me.resumeEvents();
                    }
                } 
    As I said, the patch in this thread will reduce 3 layouts to 2 layouts( same as extjs 3.4.x), it also prevents any layout due to column show/hide and grouping.
    I don't think your implementation is correct, remember the load happens asynchronously, so you'll need to resume in the load handler like:

    PHP Code:
                render: {
                    
    fn: function(){
                var 
    me this;
                   
    me.suspendEvents(); 
                   
    store.load(function () { 
                                 
    // Store load ready, resume events
                                 
    me.resumeEvents();
                    });
                    }
                } 
    But I have to admit, I did not do any testing of this myself.

  5. #15
    Ext JS Premium Member
    Join Date
    Jun 2011
    Location
    St. Louis
    Posts
    212
    Vote Rating
    9
    jimmylu98 will become famous soon enough

      0  

    Default


    Quote Originally Posted by halcwb View Post
    I don't think your implementation is correct, remember the load happens asynchronously, so you'll need to resume in the load handler like:

    PHP Code:
                render: {
                    
    fn: function(){
                var 
    me this;
                   
    me.suspendEvents(); 
                   
    store.load(function () { 
                                 
    // Store load ready, resume events
                                 
    me.resumeEvents();
                    });
                    }
                } 
    But I have to admit, I did not do any testing of this myself.
    Thank you.

    As tested in a simple example, it does have some performance improvement, but less than the patch in the proxy.

  6. #16
    Ext Premium Member halcwb's Avatar
    Join Date
    Mar 2010
    Location
    Rotterdam
    Posts
    385
    Vote Rating
    54
    halcwb is a jewel in the rough halcwb is a jewel in the rough halcwb is a jewel in the rough

      0  

    Default


    I would be very interested to have an official reply from the Sencha developers. I can't imagine that they don't have a better alternative than having to patch the internals of Extjs.