[
https://issues.apache.org/jira/browse/CASSANDRA-18717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17757551#comment-17757551
]
Maxim Muzafarov commented on CASSANDRA-18717:
---------------------------------------------
[~samt] Thank you for sharing the link to the work, the last thing we want to
do is to add another layer of complexity to the long-awaited work during the
rebase. I have my own personal experience with keeping the distributed snapshot
patch up to date for a long period of time during the review, and I remember
how painful it can be.
Now I'm confused by the question of what criteria we should use to put one of
the patches on hold and continue with the others. The number of changed files
doesn't seem to be a good one. Should we introduce such criteria and publish
them?
I have cherry-picked the fix of javadocs to the branch you provided, and
actually, it's really easy to apply. There were a few conflicts and ~8 errors
with javadoc compilation. The cause of the conflicts was mostly because the
method signature was changed e.g. TokenRingUtils#getPrimaryRangesForEndpoint,
the last ones were caused by unacceptable symbols <, > or & e.g. the javadoc
for the constant TCM_RECENTLY_SEALED_PERIOD_INDEX_SIZE. So, I think the
CASSANDRA-18314 would probably be a bigger problem to rebase on and that's why
I am confused.
The changes for your branch are here:
https://github.com/apache/cassandra/compare/cep-21-tcm...Mmuzaf:cassandra:cep-21-tcm-javadoc
> Fix errors or remove ant javadoc task
> -------------------------------------
>
> Key: CASSANDRA-18717
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18717
> Project: Cassandra
> Issue Type: Bug
> Components: Documentation/Javadoc
> Reporter: Ekaterina Dimitrova
> Assignee: Maxim Muzafarov
> Priority: Normal
> Fix For: 5.0.x, 5.x
>
> Time Spent: 50m
> Remaining Estimate: 0h
>
> As discussed on CASSANDRA-17687, the javadoc task completes successfully, but
> there are errors that deserve to be fixed if we keep the task around.
> If we do not plan to use it, we should remove it and reduce the clutter.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
