[jira] [Resolved] (ACCUMULO-3363) Deleting a property that isn't overriden in the shell prints a server error

["Christopher Tubbs (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ACCUMULO-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Christopher Tubbs resolved ACCUMULO-3363.
-----------------------------------------
       Resolution: Won't Fix
    Fix Version/s:     (was: 2.0.0)

Probably not worth adding, since it would require atomic read-modify-write operation on the server side to properly protect against this user mistake. The error message isn't super helpful, but it did behave reasonably (no change, error thrown). Since the shell is mostly an admin utility, I don't think it's a big deal. However, feel free to re-open on https://github.com/apache/accumulo/issues 

> Deleting a property that isn't overriden in the shell prints a server error
> ---------------------------------------------------------------------------
>
>                 Key: ACCUMULO-3363
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-3363
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client, shell
>    Affects Versions: 1.6.1
>            Reporter: Josh Elser
>            Priority: Major
>
> Using the shell, I tried to delete a property that didn't have an overridden value set in ZK and got an error back.
> {noformat}
> 2014-11-24 11:56:01,050 [shell.Shell] ERROR: org.apache.accumulo.core.client.AccumuloException: org.apache.thrift.TApplicationException: Internal error processing removeSystemProperty
> {noformat}
> The server didn't check to see if the node actually existed before it tried to delete it.
> {noformat}
> Problem removing config property in zookeeper
> 	org.apache.zookeeper.KeeperException$NoNodeException: KeeperErrorCode = NoNode for /accumulo/b5dce89e-6559-46bd-bd58-39a68768b2f2/config/tserver.assignment.duration.warning
> 		at org.apache.zookeeper.KeeperException.create(KeeperException.java:111)
> 		at org.apache.zookeeper.KeeperException.create(KeeperException.java:51)
> 		at org.apache.zookeeper.ZooKeeper.getChildren(ZooKeeper.java:1468)
> 		at org.apache.zookeeper.ZooKeeper.getChildren(ZooKeeper.java:1496)
> 		at org.apache.accumulo.fate.zookeeper.ZooUtil.recursiveDelete(ZooUtil.java:203)
> 		at org.apache.accumulo.fate.zookeeper.ZooReaderWriter.recursiveDelete(ZooReaderWriter.java:62)
> 		at org.apache.accumulo.server.util.SystemPropUtil.removeSystemProperty(SystemPropUtil.java:45)
> 		at org.apache.accumulo.master.MasterClientServiceHandler.removeSystemProperty(MasterClientServiceHandler.java:376)
> 		at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> 		at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
> 		at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> 		at java.lang.reflect.Method.invoke(Method.java:606)
> 		at org.apache.accumulo.trace.instrument.thrift.RpcServerInvocationHandler.invoke(RpcServerInvocationHandler.java:46)
> 		at org.apache.accumulo.server.util.RpcWrapper$1.invoke(RpcWrapper.java:44)
> 		at com.sun.proxy.$Proxy17.removeSystemProperty(Unknown Source)
> 		at org.apache.accumulo.core.master.thrift.MasterClientService$Processor$removeSystemProperty.getResult(MasterClientService.java:1392)
> 		at org.apache.accumulo.core.master.thrift.MasterClientService$Processor$removeSystemProperty.getResult(MasterClientService.java:1376)
> 		at org.apache.thrift.ProcessFunction.process(ProcessFunction.java:39)
> 		at org.apache.thrift.TBaseProcessor.process(TBaseProcessor.java:39)
> 		at org.apache.accumulo.server.util.TServerUtils$TimedProcessor.process(TServerUtils.java:168)
> 		at org.apache.thrift.server.AbstractNonblockingServer$FrameBuffer.invoke(AbstractNonblockingServer.java:516)
> 		at org.apache.accumulo.server.util.CustomNonBlockingServer$1.run(CustomNonBlockingServer.java:77)
> 		at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> 		at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> 		at org.apache.accumulo.trace.instrument.TraceRunnable.run(TraceRunnable.java:47)
> 		at org.apache.accumulo.core.util.LoggingRunnable.run(LoggingRunnable.java:34)
> 		at java.lang.Thread.run(Thread.java:745)
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)