So it is going to be a lot of work to update the schema to make it
match the spec?  I am not attached to samples in this case.  I think
having proper documentation on how to provide your own implementations
is just as valuable as a sample in this case.

On Tue, May 7, 2013 at 1:42 PM, Henry Saputra <henry.sapu...@gmail.com> wrote:
> I am trying to fix the JPA example but even the schema has changed since
> the last time any code is committed. Too much work to maintain this module
> when OpenSocial specs are moving as fast as it is right now.
>
> I am proposing to dump this java-samples module because its getting harder
> to maintain. We will rely on unit tests and sample app using the JSON
> schema.
>
> - Henry
>
>
> On Fri, May 3, 2013 at 6:19 PM, Ryan Baxter <rbaxte...@apache.org> wrote:
>
>> I don't think so.  I get test failures in that project not compile
>> errors.  Although I just checked in changes for the compiler level to
>> move it to 1.6 so you might want to update and try again.
>>
>> On Wed, May 1, 2013 at 2:55 AM, Henry Saputra <henry.sapu...@gmail.com>
>> wrote:
>> > Hmm I got compile erro when tried to run mvn clean test in java/samples
>> dir:
>> >
>> > [ERROR] Failed to execute goal
>> > org.apache.maven.plugins:maven-compiler-plugin:2.3.2:testCompile
>> > (default-testCompile) on project shindig-samples: Compilation failure
>> > [ERROR]
>> >
>> /Users/hsaputra/shindig/trunk/java/samples/src/test/java/org/apache/shindig/social/opensocial/jpa/spi/integration/JpaRestfulTestConfigHelper.java:[83,11]
>> > cannot find symbol
>> > [ERROR] symbol  : method setJSONPAllowed(boolean)
>> > [ERROR] location: class org.apache.shindig.protocol.DataServiceServlet
>> > [ERROR] -> [Help 1]
>> > [ERROR]
>> >
>> >
>> > Is this the same error?
>> >
>> > - Henry
>> >
>> >
>> > On Tue, Apr 30, 2013 at 5:23 PM, Ryan Baxter <rbaxte...@apache.org>
>> wrote:
>> >
>> >> Not really a blocker in my opinion, but I felt like we were close to
>> >> getting it in so I would at least try to make the effort ;)
>> >>
>> >> On Tue, Apr 30, 2013 at 1:32 PM, Henry Saputra <henry.sapu...@gmail.com
>> >
>> >> wrote:
>> >> > Ugh sorry for late reply. No I did not have time to look at it.
>> >> >
>> >> > I guess it is blocker for 2.5? I will take a look at it later today.
>> >> >
>> >> >
>> >> > - Henry
>> >> >
>> >> >
>> >> > On Fri, Apr 19, 2013 at 12:50 PM, Ryan Baxter <rbaxte...@gmail.com>
>> >> wrote:
>> >> >
>> >> >> Henry, have you had a sometime to look into this at all?  Would like
>> >> >> to get this into 2.5.
>> >> >>
>> >> >> On Mon, Mar 18, 2013 at 2:38 PM, Ryan Baxter <rbaxte...@gmail.com>
>> >> wrote:
>> >> >> > Yes I just uploaded the newest patch from my workspace, if you
>> have a
>> >> >> > chance take a look at the samples module.  I am just not familiar
>> >> >> > enough with JPA to know what is wrong...
>> >> >> >
>> >> >> > On Mon, Mar 18, 2013 at 2:26 PM, Henry Saputra <
>> >> henry.sapu...@gmail.com>
>> >> >> wrote:
>> >> >> >> I fixed couple issues before. So the patches for the udpated data
>> >> model
>> >> >> is
>> >> >> >> in that review entry?
>> >> >> >>
>> >> >> >> I can try it out.
>> >> >> >>
>> >> >> >> - Henry
>> >> >> >>
>> >> >> >>
>> >> >> >> On Mon, Mar 18, 2013 at 11:02 AM, Ryan Baxter <
>> rbaxte...@apache.org>
>> >> >> wrote:
>> >> >> >>
>> >> >> >>> Henry I actually have most of the 2 patches integrated together
>> [1].
>> >> >> >>> I need to some help with the JPA stuff some of the unit tests are
>> >> >> >>> failing, are you familiar with that code at all?
>> >> >> >>>
>> >> >> >>> [1] https://reviews.apache.org/r/9990/
>> >> >> >>>
>> >> >> >>> On Mon, Mar 18, 2013 at 1:46 PM, Henry Saputra <
>> >> >> henry.sapu...@gmail.com>
>> >> >> >>> wrote:
>> >> >> >>> > Yeah, I have been distracted with so many items so did not have
>> >> time
>> >> >> to
>> >> >> >>> > finish it =(
>> >> >> >>> >
>> >> >> >>> > Do you guys think it is a blocker for 2.5.0 release?
>> >> >> >>> >
>> >> >> >>> >
>> >> >> >>> > - Henry
>> >> >> >>> >
>> >> >> >>> >
>> >> >> >>> > On Sun, Mar 17, 2013 at 1:51 PM, Ryan Baxter <
>> >> rbaxte...@apache.org>
>> >> >> >>> wrote:
>> >> >> >>> >
>> >> >> >>> >> I dont think we have a list of them per-say.  Looking through
>> the
>> >> >> >>> >> JIRAs labeled with 2.5.0 I found SHINDIG-1539 [1] which tries
>> to
>> >> few
>> >> >> >>> >> some gaps in the social data objects.  I updated the patch
>> from
>> >> >> James
>> >> >> >>> >> so it builds against trunk and created a review for it [2].  I
>> >> think
>> >> >> >>> >> SHINDIG-1539 and SHINDIG-1878 are pretty much dups.  We should
>> >> apply
>> >> >> >>> >> one of the reviews and close both of them.  I am try and see
>> if
>> >> >> there
>> >> >> >>> >> are differences between the two.
>> >> >> >>> >>
>> >> >> >>> >> The other gap I can think of is the External Services section
>> in
>> >> the
>> >> >> >>> >> spec [3].  @Henry weren't you working on a patch for this?
>> >> >> >>> >>
>> >> >> >>> >> [1] https://issues.apache.org/jira/browse/SHINDIG-1539
>> >> >> >>> >> [2] https://reviews.apache.org/r/9990/
>> >> >> >>> >> [3]
>> >> >> >>> >>
>> >> >> >>>
>> >> >>
>> >>
>> http://opensocial-resources.googlecode.com/svn/spec/trunk/Core-Gadget.xml#ExternalServices
>> >> >> >>> >>
>> >> >> >>> >> On Sat, Mar 16, 2013 at 11:54 AM, Henry Saputra <
>> >> >> >>> henry.sapu...@gmail.com>
>> >> >> >>> >> wrote:
>> >> >> >>> >> > Not exactly, we have using JIRA to capture the differences.
>> >> >> Mostly the
>> >> >> >>> >> > differences come from the data model where new fields are
>> >> >> introduced
>> >> >> >>> or
>> >> >> >>> >> > have name changes.
>> >> >> >>> >> >
>> >> >> >>> >> > The big ticket AFAIK is SHINDIG-1878.
>> >> >> >>> >> >
>> >> >> >>> >> > I am +1 and reccommend for 2.5.0 release. It is been long
>> >> overdue
>> >> >> and
>> >> >> >>> >> > everybody have been putting a lot of good work.
>> >> >> >>> >> >
>> >> >> >>> >> > Thanks,
>> >> >> >>> >> >
>> >> >> >>> >> > Henr
>> >> >> >>> >> >
>> >> >> >>> >> >
>> >> >> >>> >> > On Sat, Mar 16, 2013 at 7:34 AM, Matt Franklin <
>> >> >> >>> m.ben.frank...@gmail.com
>> >> >> >>> >> >wrote:
>> >> >> >>> >> >
>> >> >> >>> >> >> On Saturday, March 16, 2013, Ryan Baxter wrote:
>> >> >> >>> >> >>
>> >> >> >>> >> >> > Hey everyone,
>> >> >> >>> >> >> >
>> >> >> >>> >> >> > 2.5.0-beta6 is out the door (thanks everyone for their
>> hard
>> >> >> work)
>> >> >> >>> and
>> >> >> >>> >> >> > I think we are at a point where 2.5.0 is pretty much
>> ready
>> >> for
>> >> >> a
>> >> >> >>> >> >> > release.  Defects have been slow to come in over the past
>> >> few
>> >> >> weeks
>> >> >> >>> >> >> > and from my point of view things are pretty stable.   I
>> know
>> >> >> there
>> >> >> >>> are
>> >> >> >>> >> >> > still some gaps between what is implemented in Shindig
>> and
>> >> >> what is
>> >> >> >>> in
>> >> >> >>> >> >> > the 2.5.0 OpenSocial spec
>> >> >> >>> >> >>
>> >> >> >>> >> >>
>> >> >> >>> >> >> do we at least have these gaps captured anywhere?
>> >> >> >>> >> >>
>> >> >> >>> >> >>
>> >> >> >>> >> >> > but I think it is also important to get a
>> >> >> >>> >> >> > non-beta release out there for other projects to consume.
>> >>  If
>> >> >> >>> everyone
>> >> >> >>> >> >> > is OK with this our next release will be the official
>> 2.5.0
>> >> >> release
>> >> >> >>> >> >> > and will not have a beta tag.
>> >> >> >>> >> >>
>> >> >> >>> >> >>
>> >> >> >>> >> >> > -Ryan
>> >> >> >>> >> >> >
>> >> >> >>> >> >>
>> >> >> >>> >>
>> >> >> >>>
>> >> >>
>> >>
>>

Reply via email to