If you have the cycles to add the validation, absolutely :)

Lars George wrote:
Duh, my bad, JM you are right, I missed to type the leading slashes...
then a little better error handling may be nice. Not sure, you guys
think this warrants a JIRA?

On Fri, Feb 3, 2017 at 3:52 PM, Ted Yu<[email protected]>  wrote:
Looking at the script:

c1.set(HConstants::ZOOKEEPER_ZNODE_PARENT, parts1[2])

It seems some validation should be added before setting the znode parent.

On Fri, Feb 3, 2017 at 5:47 AM, Jean-Marc Spaggiari<[email protected]
wrote:
Lars, I have not tested, but should it not be localhost:2181:/hbase
instead? (Note the / in the front of HBase)

JMS

2017-02-03 7:52 GMT-05:00 Lars George<[email protected]>:

I have two local instances with two different ZK ports. Note that the
error occurs already at the first call to HAdmin in the script.
Replication works find btw, it is just this script that bails.

On Fri, Feb 3, 2017 at 1:49 PM, Ted Yu<[email protected]>  wrote:
Note the port number 2182 in the second parameter.

Do you have two quorums running on the same machine ?

On Feb 3, 2017, at 3:05 AM, Lars George<[email protected]>
wrote:
Hi,

trying to run the `copy_table_desc.rb` script from the
`bin/replication` directory gives this:

```
$ bin/hbase org.jruby.Main bin/replication/copy_tables_desc.rb
localhost:2181:hbase localhost:2182:hbase-2 testtable4
...
NativeException: java.io.IOException:
java.lang.reflect.InvocationTargetException
  (root) at bin/replication/copy_tables_desc.rb:81
```

All my 5min attempts of coercing the script kiddie JRuby code to
output debug info was futile, hence leaving this here for vetting :)

Lars

Reply via email to