[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-22 Thread renato2099 
Github user renato2099 commented on the issue:

https://github.com/apache/gora/pull/97
  
thanks @vaibhavthapliyal !


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-22 Thread vaibhavthapliyal 
Github user vaibhavthapliyal commented on the issue:

https://github.com/apache/gora/pull/97
  
Cheers!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-22 Thread lewismc 
Github user lewismc commented on the issue:

https://github.com/apache/gora/pull/97
  
I adjusted my /etc/hosts file and all is good, thank you @vaibhavthapliyal  


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-22 Thread lewismc 
Github user lewismc commented on the issue:

https://github.com/apache/gora/pull/97
  
Regarding the formatting... yeah I agree. That's my fault, I thought
AccumuloStore was already formatted 2 space indents.
We should probably use the maven formatting plugin with out code template
in a subsequent issue.

On Wed, Feb 22, 2017 at 4:53 AM Renato Marroquin 
wrote:

> @lewismc  regarding the 2space vs 4space, the
> only issue with this is that now the whole AccumuloStore looks like if it
> was modified, didn't we had a style check on maven at some point? maybe we
> should have a JIRA to just make the whole code standard and keep the
> changes of this PR only addressing the ones that it's supposed to
>
> —
> You are receiving this because you were mentioned.
>
>
> Reply to this email directly, view it on GitHub
> , or mute
> the thread
> 

> .
>
-- 

*Lewis*
Dr. Lewis J. McGibbney Ph.D, B.Sc
Director, MCMA Associates
Phone: +1(626)498-3090
Skype: lewis.john.mcgibbney
Email: lewis.mcgibb...@gmail.com



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-22 Thread renato2099 
Github user renato2099 commented on the issue:

https://github.com/apache/gora/pull/97
  
@lewismc regarding the 2space vs 4space, the only issue with this is that 
now the whole AccumuloStore looks like if it was modified, didn't we had a 
style check on maven at some point? maybe we should have a JIRA to just make 
the whole code standard and keep the changes of this PR only addressing the 
ones that it's supposed to


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-22 Thread renato2099 
Github user renato2099 commented on the issue:

https://github.com/apache/gora/pull/97
  
@vaibhavthapliyal @lewismc I applied the PR and gora-accumulo passed all 
tests successfully 👍 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-22 Thread vaibhavthapliyal 
Github user vaibhavthapliyal commented on the issue:

https://github.com/apache/gora/pull/97
  
@lewismc Hi, I think the issue is with your /etc/hosts file.

Make sure you have these entries in your host file.
127.0.0.1 localhost
127.0.0.1 
Can you please check if this fixes the issue?
A reference to the solution of this can be found here:

https://community.cloudera.com/t5/Storage-Random-Access-HDFS/Problems-running-a-write-query-on-an-accumulo-cluster/td-p/21786/page/2


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-21 Thread lewismc 
Github user lewismc commented on the issue:

https://github.com/apache/gora/pull/97
  
> Do the tests proceed further in your case after this line?
No they stall... can someone else check for tests continuing and passing 
with this patch? @renato2099 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-21 Thread vaibhavthapliyal 
Github user vaibhavthapliyal commented on the issue:

https://github.com/apache/gora/pull/97
  
Hi,
Do the tests proceed further in your case after this line?
`2017-02-21 20:17:13,269 WARN  [Thrift Connection Pool Checker] 
impl.ThriftTransportPool (ThriftTransportPool.java:checkForStuckIO(206)) - 
Thread "main" stuck on IO to LMC-056430:54933 (0) for at least 120171 ms`

I get a build sucess even after the Connection Refused. I'll check for some 
fixes and revert back.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-21 Thread lewismc 
Github user lewismc commented on the issue:

https://github.com/apache/gora/pull/97
  
I also created https://issues.apache.org/jira/browse/GORA-504 to track the 
issue


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-21 Thread lewismc 
Github user lewismc commented on the issue:

https://github.com/apache/gora/pull/97
  
Hi @vaibhavthapliyal when I run the test suite, I get the following
```
---
 T E S T S
---
Running org.apache.gora.accumulo.store.AccumuloStoreTest
2017-02-21 20:15:03,855 INFO  [main] store.DataStoreTestBase 
(DataStoreTestBase.java:setUpClass(67)) - setting up class
2017-02-21 20:15:03,857 INFO  [main] gora.GoraTestDriver 
(GoraAccumuloTestDriver.java:setUpClass(50)) - Starting Accumulo 
MiniAccumuloCluster...
2017-02-21 20:15:07,283 INFO  [main] impl.MiniAccumuloClusterImpl 
(MiniAccumuloClusterImpl.java:start(609)) - Starting MAC against instance 
goraTest and zookeeper(s) localhost:56321.
2017-02-21 20:15:09,847 INFO  [main] store.DataStoreTestBase 
(DataStoreTestBase.java:setUp(90)) - setting up test
2017-02-21 20:15:10,671 WARN  [main] client.ClientConfiguration 
(ClientConfiguration.java:loadFromSearchPath(227)) - Found no client.conf in 
default paths. Using default client configuration values.
2017-02-21 20:15:10,754 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client 
environment:zookeeper.version=3.4.5-1392090, built on 09/30/2012 17:52 GMT
2017-02-21 20:15:10,755 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:host.name=lmc-056430
2017-02-21 20:15:10,755 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:java.version=1.8.0_91
2017-02-21 20:15:10,756 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:java.vendor=Oracle 
Corporation
2017-02-21 20:15:10,756 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client 
environment:java.home=/Library/Java/JavaVirtualMachines/jdk1.8.0_91.jdk/Contents/Home/jre
2017-02-21 20:15:10,757 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client 
environment:java.class.path=/usr/local/gora/gora-accumulo/target/test-classes:/usr/local/gora/gora-accumulo/target/classes
...

/Users/lmcgibbn/.m2/repository/org/apache/hadoop/hadoop-mapreduce-client-hs/2.5.2/hadoop-mapreduce-client-hs-2.5.2.jar:
2017-02-21 20:15:10,759 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client 
environment:java.library.path=/Users/lmcgibbn/Library/Java/Extensions:/Library/Java/Extensions:/Network/Library/Java/Extensions:/System/Library/Java/Extensions:/usr/lib/java:.
2017-02-21 20:15:10,760 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client 
environment:java.io.tmpdir=/var/folders/mw/f_p4r21n5ld16hnyd7j8r8w4fv8362/T/
2017-02-21 20:15:10,760 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:java.compiler=
2017-02-21 20:15:10,760 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:os.name=Mac OS X
2017-02-21 20:15:10,761 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:os.arch=x86_64
2017-02-21 20:15:10,763 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:os.version=10.11.6
2017-02-21 20:15:10,764 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:user.name=lmcgibbn
2017-02-21 20:15:10,764 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client environment:user.home=/Users/lmcgibbn
2017-02-21 20:15:10,764 INFO  [main] zookeeper.ZooKeeper 
(Environment.java:logEnv(100)) - Client 
environment:user.dir=/usr/local/gora/gora-accumulo
2017-02-21 20:15:10,765 INFO  [main] zookeeper.ZooKeeper 
(ZooKeeper.java:(438)) - Initiating client connection, 
connectString=localhost:56321 sessionTimeout=3 
watcher=org.apache.accumulo.fate.zookeeper.ZooSession$ZooWatcher@4009e306
2017-02-21 20:15:10,798 INFO  [main-SendThread(localhost:56321)] 
zookeeper.ClientCnxn (ClientCnxn.java:logStartConnect(966)) - Opening socket 
connection to server localhost/0:0:0:0:0:0:0:1:56321. Will not attempt to 
authenticate using SASL (unknown error)
2017-02-21 20:15:10,821 WARN  [main-SendThread(localhost:56321)] 
zookeeper.ClientCnxn (ClientCnxn.java:run(1089)) - Session 0x0 for server null, 
unexpected error, closing socket connection and attempting reconnect
java.net.ConnectException: Connection refused
at sun.nio.ch.SocketChannelImpl.checkConnect(Native Method)
at 
sun.nio.ch.SocketChannelImpl.finishConnect(SocketChannelImpl.java:717)
at 
org.apache.zookeeper.ClientCnxnSocketNIO.doTransport(ClientCnxnSocketNIO.java:350)
at org.apache.zookeeper.ClientCnxn$SendThread.run(ClientCnxn.java:1068)
2017-02-21 20:15:10,930 INFO  [main-SendThread(localhost:56321)] 
zookeeper.ClientCnxn (ClientCnxn.java:logStartConnect(966)) - Opening socket 
connection to server

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-21 Thread vaibhavthapliyal 
Github user vaibhavthapliyal commented on the issue:

https://github.com/apache/gora/pull/97
  
Thanks. Will be looking forward to hearing from you.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-21 Thread lewismc 
Github user lewismc commented on the issue:

https://github.com/apache/gora/pull/97
  
Yes this looks much better. I'll try and review shortly. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-18 Thread vaibhavthapliyal 
Github user vaibhavthapliyal commented on the issue:

https://github.com/apache/gora/pull/97
  
@lewismc Can you check it now, if it's formatted correctly?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] gora issue #97: Switched Accumulo Dependency to 1.7.1 and ported AccumuloSto...

2017-02-17 Thread lewismc 
Github user lewismc commented on the issue:

https://github.com/apache/gora/pull/97
  
@vaibhavthapliyal thank you for this, can you please format the code in 2 
space indents? Thank you.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---