[ https://issues.apache.org/jira/browse/HBASE-16183?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
li xiang updated HBASE-16183: ----------------------------- Description: 1. In Section 89.3.3 change {code} String path = "hdfs://<namenode>:<port>/user/<hadoop-user>/coprocessor.jar"; {code} into Path path = new Path("hdfs://bdavm1506.svl.ibm.com:8020/user/hbase/coprocessor.jar"); reason The second parameter of HTableDescriptor.addCoprocessor() is org.apache.hadoop.fs.Path, not String. See http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/HTableDescriptor.html 2. In Section 89.3.3 change HBaseAdmin admin = new HBaseAdmin(conf); into Connection connection = ConnectionFactory.createConnection(conf); Admin admin = connection.getAdmin(); reason HBASE-12083 makes new HBaseAdmin() deprecated and the instance of Admin is supposed to get from Connection.getAdmin() Also see http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/client/HBaseAdmin.html 3. In section 90.1 change public void preGetOp(final ObserverContext e, final Get get, final List results) into public void preGetOp(final ObserverContext<RegionCoprocessorEnvironment> e, final Get get, final List<Cell> results) change List kvs = new ArrayList(results.size()); into List<Cell> kvs = new ArrayList<Cell>(results.size()); change public RegionScanner preScannerOpen(final ObserverContext e, final Scan scan, into preScannerOpen(final ObserverContext<RegionCoprocessorEnvironment> e, final Scan scan, change public boolean postScannerNext(final ObserverContext e, final InternalScanner s, final List results, final int limit, final boolean hasMore) throws IOException { into public boolean postScannerNext(final ObserverContext<RegionCoprocessorEnvironment> e, final InternalScanner s, inal List<Result> results, final int limit, final boolean hasMore) throws IOException { change Iterator iterator = results.iterator(); into Iterator<Result> iterator = results.iterator(); reason Generic 4. In section 90.1 change preGet(e, get, kvs); into super.preGetOp(e, get, kvs); reason There is not a function called preGet() provided by BaseRegionObserver or its super class/interface. I believe we need to call preGetOp() of the super class of RegionObserverExample here. 5. In section 90.1 change kvs.add(KeyValueUtil.ensureKeyValue(c)); into kvs.add(c); reason KeyValueUtil.ensureKeyValue() is deprecated. See http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/KeyValueUtil.html and https://issues.apache.org/jira/browse/HBASE-12079 was: 1. In Section 89.3.3 change String path = "hdfs://<namenode>:<port>/user/<hadoop-user>/coprocessor.jar"; into Path path = new Path("hdfs://bdavm1506.svl.ibm.com:8020/user/hbase/coprocessor.jar"); reason The second parameter of HTableDescriptor.addCoprocessor() is org.apache.hadoop.fs.Path, not String. See http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/HTableDescriptor.html 2. In Section 89.3.3 change HBaseAdmin admin = new HBaseAdmin(conf); into Connection connection = ConnectionFactory.createConnection(conf); Admin admin = connection.getAdmin(); reason HBASE-12083 makes new HBaseAdmin() deprecated and the instance of Admin is supposed to get from Connection.getAdmin() Also see http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/client/HBaseAdmin.html 3. In section 90.1 change public void preGetOp(final ObserverContext e, final Get get, final List results) into public void preGetOp(final ObserverContext<RegionCoprocessorEnvironment> e, final Get get, final List<Cell> results) change List kvs = new ArrayList(results.size()); into List<Cell> kvs = new ArrayList<Cell>(results.size()); change public RegionScanner preScannerOpen(final ObserverContext e, final Scan scan, into preScannerOpen(final ObserverContext<RegionCoprocessorEnvironment> e, final Scan scan, change public boolean postScannerNext(final ObserverContext e, final InternalScanner s, final List results, final int limit, final boolean hasMore) throws IOException { into public boolean postScannerNext(final ObserverContext<RegionCoprocessorEnvironment> e, final InternalScanner s, inal List<Result> results, final int limit, final boolean hasMore) throws IOException { change Iterator iterator = results.iterator(); into Iterator<Result> iterator = results.iterator(); reason Generic 4. In section 90.1 change preGet(e, get, kvs); into super.preGetOp(e, get, kvs); reason There is not a function called preGet() provided by BaseRegionObserver or its super class/interface. I believe we need to call preGetOp() of the super class of RegionObserverExample here. 5. In section 90.1 change kvs.add(KeyValueUtil.ensureKeyValue(c)); into kvs.add(c); reason KeyValueUtil.ensureKeyValue() is deprecated. See http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/KeyValueUtil.html and https://issues.apache.org/jira/browse/HBASE-12079 > Correct errors in example program of coprocessor in Ref Guide > ------------------------------------------------------------- > > Key: HBASE-16183 > URL: https://issues.apache.org/jira/browse/HBASE-16183 > Project: HBase > Issue Type: Bug > Components: documentation > Reporter: li xiang > Assignee: li xiang > Priority: Minor > > 1. In Section 89.3.3 > change > {code} > String path = "hdfs://<namenode>:<port>/user/<hadoop-user>/coprocessor.jar"; > {code} > into > Path path = new > Path("hdfs://bdavm1506.svl.ibm.com:8020/user/hbase/coprocessor.jar"); > reason > The second parameter of HTableDescriptor.addCoprocessor() is > org.apache.hadoop.fs.Path, not String. > See > http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/HTableDescriptor.html > 2. In Section 89.3.3 > change > HBaseAdmin admin = new HBaseAdmin(conf); > into > Connection connection = ConnectionFactory.createConnection(conf); > Admin admin = connection.getAdmin(); > reason > HBASE-12083 makes new HBaseAdmin() deprecated and the instance of Admin is > supposed to get from Connection.getAdmin() > Also see > http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/client/HBaseAdmin.html > 3. In section 90.1 > change > public void preGetOp(final ObserverContext e, final Get get, final List > results) > into > public void preGetOp(final ObserverContext<RegionCoprocessorEnvironment> e, > final Get get, final List<Cell> results) > change > List kvs = new ArrayList(results.size()); > into > List<Cell> kvs = new ArrayList<Cell>(results.size()); > change > public RegionScanner preScannerOpen(final ObserverContext e, final Scan > scan, > into > preScannerOpen(final ObserverContext<RegionCoprocessorEnvironment> e, final > Scan scan, > change > public boolean postScannerNext(final ObserverContext e, final > InternalScanner s, > final List results, final int limit, final boolean hasMore) throws > IOException { > into > public boolean postScannerNext(final > ObserverContext<RegionCoprocessorEnvironment> e, final InternalScanner s, > inal List<Result> results, final int limit, final boolean hasMore) throws > IOException { > change > Iterator iterator = results.iterator(); > into > Iterator<Result> iterator = results.iterator(); > reason > Generic > 4. In section 90.1 > change > preGet(e, get, kvs); > into > super.preGetOp(e, get, kvs); > reason > There is not a function called preGet() provided by BaseRegionObserver or > its super class/interface. I believe we need to call preGetOp() of the super > class of RegionObserverExample here. > 5. In section 90.1 > change > kvs.add(KeyValueUtil.ensureKeyValue(c)); > into > kvs.add(c); > reason > KeyValueUtil.ensureKeyValue() is deprecated. > See > http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/KeyValueUtil.html > and https://issues.apache.org/jira/browse/HBASE-12079 -- This message was sent by Atlassian JIRA (v6.3.4#6332)