You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@rya.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/08/14 19:29:00 UTC

[jira] [Commented] (RYA-343) AccumuloLoadStatementsFile fails when loading data to a PCJ-enabled table.

    [ https://issues.apache.org/jira/browse/RYA-343?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16126273#comment-16126273 ] 

ASF GitHub Bot commented on RYA-343:
------------------------------------

GitHub user jdasch opened a pull request:

    https://github.com/apache/incubator-rya/pull/205

    RYA-343 Fixed rya.api connection issue for PCJ-enabled rya table.

    <!--
    Licensed to the Apache Software Foundation (ASF) under one
    or more contributor license agreements.  See the NOTICE file
    distributed with this work for additional information
    regarding copyright ownership.  The ASF licenses this file
    to you under the Apache License, Version 2.0 (the
    "License"); you may not use this file except in compliance
    with the License.  You may obtain a copy of the License at
    
      http://www.apache.org/licenses/LICENSE-2.0
    
    Unless required by applicable law or agreed to in writing,
    software distributed under the License is distributed on an
    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    KIND, either express or implied.  See the License for the
    specific language governing permissions and limitations
    under the License.
    -->
    ## Description
    Added a couple configs that weren't getting set and caused a regression.
    
    Purpose of this PR is to get a fix in place now.  We should probably make a new JIRA to ensure this regression is captured by an IT and to expand PCJDetails to store/retrieve this information.
    
    ### Tests
    None.
    
    ### Links
    [Jira: RYA-343](https://issues.apache.org/jira/browse/RYA-343)
    
    ### Checklist
    - [ ] Code Review
    - [ ] Squash Commits
    
    #### People To Reivew
    @amihalik 
    @meiercaleb 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/jdasch/incubator-rya RYA-343

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-rya/pull/205.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #205
    
----
commit 81477c5815f3f54a3ccd5d32e87e6692461b32a9
Author: jdasch <jd...@localhost.localdomain>
Date:   2017-08-14T19:21:10Z

    RYA-343 Fixed rya.api connection issue for PCJ-enabled rya table.

----


> AccumuloLoadStatementsFile fails when loading data to a PCJ-enabled table.
> --------------------------------------------------------------------------
>
>                 Key: RYA-343
>                 URL: https://issues.apache.org/jira/browse/RYA-343
>             Project: Rya
>          Issue Type: Sub-task
>          Components: clients
>            Reporter: Jeff Dasch
>            Assignee: Jeff Dasch
>
> Issue occurs when calling {{AccumuloLoadStatementsFile.loadStatements()}} to loading data to a PCJ-enabled table.  I believe this is a recent regression.
> {noformat}
> 2017-08-14 13:46:51,802 [Spring Shell] WARN  org.apache.rya.api.client.accumulo.AccumuloLoadStatementsFile - Exception while loading:
> org.apache.rya.api.persist.RyaDAOException: java.lang.IllegalArgumentException: The 'rya.indexing.pcj.storageType' property must have one of the following values: [ACCUMULO]
> 	at org.apache.rya.accumulo.AccumuloRyaDAO.init(AccumuloRyaDAO.java:165)
> 	at org.apache.rya.sail.config.RyaSailFactory.getAccumuloDAO(RyaSailFactory.java:155)
> 	at org.apache.rya.sail.config.RyaSailFactory.getRyaSail(RyaSailFactory.java:100)
> 	at org.apache.rya.sail.config.RyaSailFactory.getInstance(RyaSailFactory.java:67)
> 	at org.apache.rya.api.client.accumulo.AccumuloLoadStatementsFile.loadStatements(AccumuloLoadStatementsFile.java:91)
> 	at org.apache.rya.shell.RyaCommands.loadData(RyaCommands.java:121)
> 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> 	at java.lang.reflect.Method.invoke(Method.java:498)
> 	at org.springframework.util.ReflectionUtils.invokeMethod(ReflectionUtils.java:210)
> 	at org.springframework.shell.core.SimpleExecutionStrategy.invoke(SimpleExecutionStrategy.java:64)
> 	at org.springframework.shell.core.SimpleExecutionStrategy.execute(SimpleExecutionStrategy.java:57)
> 	at org.springframework.shell.core.AbstractShell.executeCommand(AbstractShell.java:127)
> 	at org.springframework.shell.core.JLineShell.promptLoop(JLineShell.java:533)
> 	at org.springframework.shell.core.JLineShell.run(JLineShell.java:179)
> 	at java.lang.Thread.run(Thread.java:748)
> Caused by: java.lang.IllegalArgumentException: The 'rya.indexing.pcj.storageType' property must have one of the following values: [ACCUMULO]
> 	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:92)
> 	at org.apache.rya.indexing.external.PrecomputedJoinStorageSupplier.get(PrecomputedJoinStorageSupplier.java:68)
> 	at org.apache.rya.indexing.external.PrecomputedJoinIndexer.init(PrecomputedJoinIndexer.java:139)
> 	at org.apache.rya.accumulo.AccumuloRyaDAO.init(AccumuloRyaDAO.java:156)
> 	... 16 more
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)