Elias Torres wrote:
Guys,
I have committed my initial coding in tagging and believe have gotten
quiet far on this first pass.
http://svn.apache.org/viewvc?view=rev&revision=448460
If you have a chance to try it, you should be able to do the following:
- Create a new entry with tags
- Edit an existing entry and add tags
- Frontpage should display tags in the entry summary
- A new tab "Site Tags" in the frontpage will give you a nice cloud
- A Hot Tags section on the right
- /roller/frontpageblog/tags/tag1+tag2 should work.
The part that got really confusing to me was the pager part. I
definitely need guidance here on what is the right way to go. I'm also
confused at the moment on caching. It seems like the frontpage is
caching anything as the front page. I just noticed all of this last once
I finally got a basic pager working, but it's a start. Please start
reviewing and tell me what parts of the code are too hacky and need to
be cleaned up.
cool, that's definitely a good start.
I think we may have goofed a little on one part of the proposal though,
which is the getWeblogEntriesByTags() method in WeblogManager. I think
that instead of adding that method what we really want to do is update
the existing getWeblogEntriesXXXMap() methods to support tags. The
problem with the existing method is that it returns a list, which is not
really how we do things right now, instead we return a map which has
sorted the result set by time and keyed each map entry by date. i.e. so
that each map entry is a date with a list of entries from that date.
Once we do that then I think the pager part is easier. I don't think
you need your own pager actually, I think you can just make sure that
the existing pagers are able to make use of tags. That way when someone
uses the url /tags/foo+bar you are just using the LatestPager with a
constraint to a set of tags.
Caching is another big consideration which wasn't addressed in the
proposal at all, so we'll still have to sort that out. A basic rundown
of caching goes like this ...
there are 3 rendering caches: sitewide, weblog pages, weblog feeds.
everything from the sitewide weblog goes into that cache because it has
special considerations with the way it expires content. everything else
goes into the page and feed caches.
everything that is cached for a weblog is based off of the
weblog.lastModified attribute. if that attribute is updated then all
the content for a weblog is expired and will be rerendered on the next
request.
content is put in the caches in the same way for all 3 caches and only
the page and feed servlets use the caches. the process is basically ...
1. parse request into a version of XXXWeblogRequest object.
2. build cache key based on XXXWeblogRequest object.
3. check if cached content exists and is still valid.
4. render page (if needed)
5. cache it
you can look in the WeblogPageCache and WeblogFeedCache objects to see
how the cache key generation works, it basically just inspects the
attributes in the XXXWeblogRequest object and builds a unique string out
of it. you will definitely need to update that to make sure the cache
keys work for tags.
the tricky part now is figuring out how we actually want the caching to
work for tags. this is something that should probably be configurable
if at all possible. i can see people wanted to completely disable
caching for tags (possibly default?), enable it for single tags only, etc.
let me know if you have more questions about it.
-- Allen
Happy tagging!
-Elias
