FSA Support crawl-time DisplayURL

FSA Support crawl-time DisplayURL

[pjo, 2013-11-28 01:32:08]

Nice.  LGTM.  Thank you.


- technology's compounding interest
-


On Wed, Nov 27, 2013 at 5:24 PM, <Brett.Michael.Johnson@gmail.com> wrote:

> Reviewers: pjo, ejona, mifern,
>
> Description:
> FSA Support crawl-time DisplayURL
>
> Please review this at http://codereview.appspot.com/34560043/
>
> Affected files (+1, -0 lines):
>   M src/com/google/enterprise/adaptor/fs/FsAdaptor.java
>
>
> Index: src/com/google/enterprise/adaptor/fs/FsAdaptor.java
> diff --git a/src/com/google/enterprise/adaptor/fs/FsAdaptor.java
> b/src/com/google/enterprise/adaptor/fs/FsAdaptor.java
> index 23d1a6c98e0903c25da719c70f8932f2390ed144..
> 090ea7912d75fd17937f0ccdca66297b30a169eb 100644
> --- a/src/com/google/enterprise/adaptor/fs/FsAdaptor.java
> +++ b/src/com/google/enterprise/adaptor/fs/FsAdaptor.java
> @@ -312,6 +312,7 @@ public class FsAdaptor extends AbstractAdaptor {
>          BasicFileAttributes.class, LinkOption.NOFOLLOW_LINKS);
>      final FileTime lastAccessTime = attrs.lastAccessTime();
>
> +    resp.setDisplayUrl(doc.toUri());
>      resp.setLastModified(new Date(attrs.lastModifiedTime().toMillis()));
>      resp.addMetadata("Creation Time", dateFormatter.get().format(
>          new Date(attrs.creationTime().toMillis())));
>
>
>

[ejona, 2013-12-02 15:40:24]

LGTM

[Brett, 2013-12-06 01:56:45]

Committed 27 November 2013 to File System Adaptor