feat(bigtable): Remove deprecated Bytes from BigEndianBytesEncoding

ron-gal commented 1 month ago

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)
[x] Rollback plan is reviewed and LGTMed
[x] All new data plane features have a completed end to end testing plan

Fixes # ☕️

If you write sample code, please follow the samples format.

steveniemitz commented 1 month ago

/gcbrun

steveniemitz commented 1 month ago

similar to my comment on the go side, I'm fine with an API breaking change here:

This is still @BetaAPI (we should remove that though)
It has only existed for ~a week
Its unlikely anyone is using the more complicated builder API that we changed here anyways

steveniemitz commented 1 month ago

/gcbrun

googleapis / java-bigtable