Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(769)

Issue 1199044: Un-deprecate nonpaged RestfullCollection

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 11 months ago by zhoresh
Modified:
13 years, 11 months ago
Reviewers:
Paul Lindner, shindig.remailer
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk
Visibility:
Public.

Description

Add back the constructor which is just a helper constructor to create RestfullCollection without paging.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
java/common/src/main/java/org/apache/shindig/protocol/RestfulCollection.java View 1 chunk +4 lines, -0 lines 1 comment Download

Messages

Total messages: 3
zhoresh
13 years, 11 months ago (2010-05-15 01:31:05 UTC) #1
Paul Lindner
http://codereview.appspot.com/1199044/diff/1/2 File java/common/src/main/java/org/apache/shindig/protocol/RestfulCollection.java (left): http://codereview.appspot.com/1199044/diff/1/2#oldcode62 java/common/src/main/java/org/apache/shindig/protocol/RestfulCollection.java:62: maybe add some javadoc here.. Add warnings about not ...
13 years, 11 months ago (2010-05-15 06:35:24 UTC) #2
zhoresh
13 years, 11 months ago (2010-05-15 15:59:13 UTC) #3
On 2010/05/15 06:35:24, Paul Lindner wrote:
> http://codereview.appspot.com/1199044/diff/1/2
> File
> java/common/src/main/java/org/apache/shindig/protocol/RestfulCollection.java
> (left):
> 
> http://codereview.appspot.com/1199044/diff/1/2#oldcode62
>
java/common/src/main/java/org/apache/shindig/protocol/RestfulCollection.java:62:
> 
> maybe add some javadoc here..  Add warnings about not using in a paginated
> context..

Javadoc added and committed r944672
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b