Skip to content

HBASE-29878 Add missing footnotes (addendum)#7878

Merged
PDavid merged 5 commits intoapache:masterfrom
yuriipalam:HBASE-29878-add-missing-footnotes
Mar 9, 2026
Merged

HBASE-29878 Add missing footnotes (addendum)#7878
PDavid merged 5 commits intoapache:masterfrom
yuriipalam:HBASE-29878-add-missing-footnotes

Conversation

@yuriipalam
Copy link
Contributor

The new docs had some missing footnotes, this PR adds them back. Also, I added some testing to ensure the uniqueness of all reference ids across all the files.

@yuriipalam yuriipalam requested a review from PDavid March 6, 2026 16:48
Copy link
Contributor

@PDavid PDavid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many thanks! 👍

Special thanks for extending the documentation and the tests! 💪

[^1]: Running an offline upgrade tool without downgrade might be needed. We will typically only support migrating data from major version X to major version X+1.
[^1]: See 'Source Compatibility' https://wiki.openjdk.org/spaces/csr/pages/32342052/Kinds+of+Compatibility

[^2]: See http://docs.oracle.com/javase/specs/jls/se7/html/jls-13.html.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I mentioned here https://issues.apache.org/jira/browse/HBASE-29957?filter=12352185#:~:text=Additionally%2C%20I%20noticed,8%20or%2017)%EF%BC%9F,
The link we are using is for Java SE 7. Isn't this version a bit outdated?
I suggest updating it to SE 8. What do you think? @PDavid @yuriipalam

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I think this makes perfect sense. 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @guluo2016
Done.

@PDavid PDavid requested a review from guluo2016 March 9, 2026 11:44
Copy link
Member

@guluo2016 guluo2016 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@PDavid PDavid merged commit 8af9d1c into apache:master Mar 9, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants