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

Issue 108400046: LIB Do not send ACLs and Groups to GSA if markAllDocsAsPublic is true. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 9 months ago by Brett
Modified:
9 years, 9 months ago
Reviewers:
pjo
CC:
connector-cr_google.com
Visibility:
Public.

Description

If markAllDocsAsPublic is true, feeding ACLs and Groups serves no purpose. This modifies DocIdSender to return immediately if pushNamedResources() or pushGroupDefinitions() are called when markAllDocsAsPublic is true. When I added tests for this, I found some of the tests for pushing named resources and groups normally were deficient, so I beefed them up a bit.

Patch Set 1 #

Total comments: 4

Patch Set 2 : Add logging. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -1 line) Patch
M src/com/google/enterprise/adaptor/DocIdSender.java View 1 2 chunks +10 lines, -0 lines 0 comments Download
M test/com/google/enterprise/adaptor/DocIdSenderTest.java View 2 chunks +49 lines, -1 line 0 comments Download

Messages

Total messages: 6
Brett
9 years, 9 months ago (2014-07-03 23:21:24 UTC) #1
pjo
LGTM. Thank you https://codereview.appspot.com/108400046/diff/1/src/com/google/enterprise/adaptor/DocIdSender.java File src/com/google/enterprise/adaptor/DocIdSender.java (right): https://codereview.appspot.com/108400046/diff/1/src/com/google/enterprise/adaptor/DocIdSender.java#newcode146 src/com/google/enterprise/adaptor/DocIdSender.java:146: return null; add FINEST log? https://codereview.appspot.com/108400046/diff/1/src/com/google/enterprise/adaptor/DocIdSender.java#newcode252 ...
9 years, 9 months ago (2014-07-07 17:15:17 UTC) #2
Brett
https://codereview.appspot.com/108400046/diff/1/src/com/google/enterprise/adaptor/DocIdSender.java File src/com/google/enterprise/adaptor/DocIdSender.java (right): https://codereview.appspot.com/108400046/diff/1/src/com/google/enterprise/adaptor/DocIdSender.java#newcode146 src/com/google/enterprise/adaptor/DocIdSender.java:146: return null; On 2014/07/07 17:15:17, pjo wrote: > add ...
9 years, 9 months ago (2014-07-08 21:18:24 UTC) #3
Brett
Add logging.
9 years, 9 months ago (2014-07-08 21:20:10 UTC) #4
pjo
LGTM. Thank you.
9 years, 9 months ago (2014-07-08 21:57:50 UTC) #5
Brett
9 years, 9 months ago (2014-07-08 22:01:29 UTC) #6
Committed 08 July 2014 to Adaptor Library:

To https://code.google.com/p/plexi/
   3c40f45..0fd57ed  master -> master
Sign in to reply to this message.

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