[jira] Created: (TAPESTRY-2502) ridDataSource and possible 'race condition'

[Tapestry - Dev mailing list]

ridDataSource and possible 'race condition'
-------------------------------------------

                 Key: TAPESTRY-2502
                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2502
             Project: Tapestry
          Issue Type: Bug
          Components: Core Components
    Affects Versions: 5.0.13
            Reporter: Martijn Brinkers


GridDataSource requires you to return the number of rows from the
getAvailableRows method. Later prepare gets called allowing you to get
the records (or some of). Now if between the call to getAvailableRows
and prepare some records are deleted you'll end up with less records
then reported by getAvailableRows. Now what should be returned from
getRowValue? I tried returning null but that was not allowed (like in
exceptions occured).

What I would like is that when getRowValue returns
null that the grid stops requesting more rows. This would also make it
easier when you do not know how many rows you have in advance.


--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Tapestry - Dev mailing list]

     [ https://issues.apache.org/jira/browse/TAPESTRY-2502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Martijn Brinkers updated TAPESTRY-2502:
---------------------------------------

    Summary: gridDataSource and possible 'race condition'  (was: ridDataSource and possible 'race condition')

> gridDataSource and possible 'race condition'
> --------------------------------------------
>
>                 Key: TAPESTRY-2502
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2502
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Core Components
>    Affects Versions: 5.0.13
>            Reporter: Martijn Brinkers
>
> GridDataSource requires you to return the number of rows from the
> getAvailableRows method. Later prepare gets called allowing you to get
> the records (or some of). Now if between the call to getAvailableRows
> and prepare some records are deleted you'll end up with less records
> then reported by getAvailableRows. Now what should be returned from
> getRowValue? I tried returning null but that was not allowed (like in
> exceptions occured).
> What I would like is that when getRowValue returns
> null that the grid stops requesting more rows. This would also make it
> easier when you do not know how many rows you have in advance.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Tapestry - Dev mailing list]

    [ https://issues.apache.org/jira/browse/TAPESTRY-2502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12611311#action_12611311 ]

Martijn Brinkers commented on TAPESTRY-2502:
--------------------------------------------

As a workaround you can return an 'empty' row element and set the rowClass for this row to make the row invisible.

Example:

public String getRowClass()
{
  String rowClass = null;

  if (certificate.isNull())
  {
    /*
     * It's an empty row.
     */
    rowClass = nullRowClass;
  }

  return rowClass;
}

and nullRowClass something like:

.empty-element {
        display : none;
}

> gridDataSource and possible 'race condition'
> --------------------------------------------
>
>                 Key: TAPESTRY-2502
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2502
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Core Components
>    Affects Versions: 5.0.13
>            Reporter: Martijn Brinkers
>
> GridDataSource requires you to return the number of rows from the
> getAvailableRows method. Later prepare gets called allowing you to get
> the records (or some of). Now if between the call to getAvailableRows
> and prepare some records are deleted you'll end up with less records
> then reported by getAvailableRows. Now what should be returned from
> getRowValue? I tried returning null but that was not allowed (like in
> exceptions occured).
> What I would like is that when getRowValue returns
> null that the grid stops requesting more rows. This would also make it
> easier when you do not know how many rows you have in advance.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Tapestry - Dev mailing list]

     [ https://issues.apache.org/jira/browse/TAPESTRY-2502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Howard M. Lewis Ship reassigned TAPESTRY-2502:
----------------------------------------------

    Assignee: Howard M. Lewis Ship

> gridDataSource and possible 'race condition'
> --------------------------------------------
>
>                 Key: TAPESTRY-2502
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2502
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Core Components
>    Affects Versions: 5.0.13
>            Reporter: Martijn Brinkers
>            Assignee: Howard M. Lewis Ship
>
> GridDataSource requires you to return the number of rows from the
> getAvailableRows method. Later prepare gets called allowing you to get
> the records (or some of). Now if between the call to getAvailableRows
> and prepare some records are deleted you'll end up with less records
> then reported by getAvailableRows. Now what should be returned from
> getRowValue? I tried returning null but that was not allowed (like in
> exceptions occured).
> What I would like is that when getRowValue returns
> null that the grid stops requesting more rows. This would also make it
> easier when you do not know how many rows you have in advance.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Tapestry - Dev mailing list]

     [ https://issues.apache.org/jira/browse/TAPESTRY-2502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Howard M. Lewis Ship updated TAPESTRY-2502:
-------------------------------------------

    Summary: When the GridDataSource reports more available rows than actual rows (due to race conditions with updates), the Grid will fail because of IndexOutOfRange exceptions  (was: gridDataSource and possible 'race condition')

> When the GridDataSource reports more available rows than actual rows (due to race conditions with updates), the Grid will fail because of IndexOutOfRange exceptions
> --------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-2502
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2502
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Core Components
>    Affects Versions: 5.0.13
>            Reporter: Martijn Brinkers
>            Assignee: Howard M. Lewis Ship
>             Fix For: 5.0.14
>
>
> GridDataSource requires you to return the number of rows from the
> getAvailableRows method. Later prepare gets called allowing you to get
> the records (or some of). Now if between the call to getAvailableRows
> and prepare some records are deleted you'll end up with less records
> then reported by getAvailableRows. Now what should be returned from
> getRowValue? I tried returning null but that was not allowed (like in
> exceptions occured).
> What I would like is that when getRowValue returns
> null that the grid stops requesting more rows. This would also make it
> easier when you do not know how many rows you have in advance.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...

[Tapestry - Dev mailing list]

     [ https://issues.apache.org/jira/browse/TAPESTRY-2502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Howard M. Lewis Ship closed TAPESTRY-2502.
------------------------------------------

       Resolution: Fixed
    Fix Version/s: 5.0.14

> When the GridDataSource reports more available rows than actual rows (due to race conditions with updates), the Grid will fail because of IndexOutOfRange exceptions
> --------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-2502
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2502
>             Project: Tapestry
>          Issue Type: Bug
>          Components: Core Components
>    Affects Versions: 5.0.13
>            Reporter: Martijn Brinkers
>            Assignee: Howard M. Lewis Ship
>             Fix For: 5.0.14
>
>
> GridDataSource requires you to return the number of rows from the
> getAvailableRows method. Later prepare gets called allowing you to get
> the records (or some of). Now if between the call to getAvailableRows
> and prepare some records are deleted you'll end up with less records
> then reported by getAvailableRows. Now what should be returned from
> getRowValue? I tried returning null but that was not allowed (like in
> exceptions occured).
> What I would like is that when getRowValue returns
> null that the grid stops requesting more rows. This would also make it
> easier when you do not know how many rows you have in advance.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@...
For additional commands, e-mail: dev-help@...