Community
Participate
Working Groups
Comments posted on the left side of the compare editor show up on right side after submission. E.g. if I post a comment on line 100 of the base revision, it shows up on line 100 of the right.
Comment drafts that are submitted to Gerrit have a "side" attribute associated with them. In Mylyn, this side attribute is only set to the "PARENT" or left side in a comment for the base revision. So the way Gerrit deals with comment publishing is that it gets all of the drafts first and then makes a POST to set the review. In our code, the draft getter is able to retrieve all of the info about the comment drafts (represented as CommentInfo) from Gerrit (including the "side" attribute). However, the set review POST call requires CommentInputs, so we convert those CommentInfos. In our conversion, we never set the "side" attribute, so it is omitted. Gerrit assumes that all comments without a side attribute are comments for the revision side (https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#comment-input). This explains why it was always posting base left side comments to the right side.
Thanks Chris! I've pushed a fix to https://git.eclipse.org/r/#/c/48503/. I don't know why we were converting from CommentInfos to CommentInputs since a CommentInfo is already a CommentInput, but I've just removed the CommentInfo class entirely since it seems unneeded. Can you add a test in a separate review? There are tests of inline comments in PatchSetRemoteFactoryTest and it's easy enough to publish comments on the left side (even if the file doesn't exist in teh base), but I'm not sure how to retrieve them and check that they're on the correct side.
Added some tests in this review - https://git.eclipse.org/r/#/c/48577/. This should ensure that the comments are posted on the right revision (they fail in a build before the change, and pass after).
Thanks. The changes have been merged.
