Skip to content

Conversation

@jojochuang
Copy link
Contributor

…yncwal (#1301) (#1535)

  • Use Reflection to access shaded Hadoop protobuf classes.

(cherry picked from commit a321e536989083ca3620bf2c53f12c07740bf5b0)

  • Update to improve the code:
  1. Added license.
  2. Added more comments.
  3. Wrap byte array instead of copy to make a ByteString.
  4. Moved all reflection instantiation to static class loading time.
  • Use LiteralByteString to wrap byte array instead of copying it.

Signed-off-by: Duo Zhang [email protected]
Signed-off-by: stack [email protected]
(cherry picked from commit 72727ff)
(cherry picked from commit ae3de38)
(cherry picked from commit b8b8e0a)

…yncwal (apache#1301) (apache#1535)

* Use Reflection to access shaded Hadoop protobuf classes.

(cherry picked from commit a321e536989083ca3620bf2c53f12c07740bf5b0)

* Update to improve the code:

1. Added license.
2. Added more comments.
3. Wrap byte array instead of copy to make a ByteString.
4. Moved all reflection instantiation to static class loading time.

* Use LiteralByteString to wrap byte array instead of copying it.

Signed-off-by: Duo Zhang <[email protected]>
Signed-off-by: stack <[email protected]>
(cherry picked from commit 72727ff)
(cherry picked from commit ae3de38)
(cherry picked from commit b8b8e0a)
@jojochuang
Copy link
Contributor Author

Clean backport. Open this PR to let it run through precommit.

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ docker 2m 34s Docker failed to build yetus/hbase:d9b4982ad4.
Subsystem Report/Notes
GITHUB PR #1567
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1567/1/console
versions git=2.17.1
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@saintstack saintstack merged commit 6a2314d into apache:branch-2.2 Apr 29, 2020
wchevreuil pushed a commit to wchevreuil/hbase that referenced this pull request May 24, 2021
…yncwal (apache#1301) (apache#1535) (apache#1567)

* Use Reflection to access shaded Hadoop protobuf classes.

(cherry picked from commit a321e536989083ca3620bf2c53f12c07740bf5b0)

* Update to improve the code:

1. Added license.
2. Added more comments.
3. Wrap byte array instead of copy to make a ByteString.
4. Moved all reflection instantiation to static class loading time.

* Use LiteralByteString to wrap byte array instead of copying it.

Signed-off-by: Duo Zhang <[email protected]>
Signed-off-by: stack <[email protected]>
(cherry picked from commit 72727ff)
(cherry picked from commit ae3de38)
(cherry picked from commit b8b8e0a)
(cherry picked from commit 6a2314d)

Change-Id: Id87f30e022d36acbd629364e01e952954cce4460
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.

3 participants