hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Created] (HBASE-18790) Purge all remaining Writables from the codebase
Date Mon, 11 Sep 2017 18:45:01 GMT
stack created HBASE-18790:

             Summary: Purge all remaining Writables from the codebase
                 Key: HBASE-18790
                 URL: https://issues.apache.org/jira/browse/HBASE-18790
             Project: HBase
          Issue Type: Task
          Components: Protobufs
            Reporter: stack

Did a review on Writables in hbase. We have the following still in HBase.  This issue is about
purging all

$ grep -r -e 'implements Writable' -e 'extends VersionedWritable' hbase-*/src/main/java

class Permission extends VersionedWritable {
WritableComparable<ImmutableBytesWritable> {
hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/mapreduce/Import.java:      implements
WritableComparable<KeyValueWritableComparable> {
 public static class TableSnapshotRegionSplit extends InputSplit implements Writable {
 public static class InputSplit implements Writable {
Writable, Comparable<TableSplit> {
hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/mapreduce/WALInputFormat.java:  static
class WALSplit extends InputSplit implements Writable {
hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java:    private
static class ExportSnapshotInputSplit extends InputSplit implements Writable {
 private class MetaWriter implements Writable {
class TimeRangeTracker implements Writable {
class AuthenticationKey implements Writable {

HBASE-18754 is about getting rid of Writable from TimeRangeTracker.

For TablePermission/Permission/UserPermission, we read permissions from acl table on initialization
of accesscontroller in the loadAll method. We scan the acl and load up a Map of perms. We
don't use the TablePermission Writable deserializing here; we just serialize the tablename
raw bytes as part of row key. It is hard to follow what is going on -- comments don't agree
w/ code -- and it needs some cleanup but seems like Writables are unused in Permission and

This message was sent by Atlassian JIRA

View raw message