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

ASF GitHub Bot commented on JENA-1083:
--------------------------------------

GitHub user ajs6f opened a pull request:

    https://github.com/apache/jena/pull/110

    JENA-1083: Using ColumnMap in transaction in-memory dataset implementation

    Okay, @afs : Second try. This time I avoid unneeded object creation or 
packing and unpacking, and I don't do weird things to the semantics of `Quad` 
and `Triple` (I hope!).
    
    I did add some convenience methods to `ColumnMap` and also let the two 
tuple types subtype `Tuple<Node>` in order to work more closely with 
`ColumnMap`. I don't think this should be any problem, since they truly are 
tuples, but I'm sure you will let me know if that is not the case :).
    
    The gains here are much more clarity for the forms of 
`PMapTupleTable`-based in-memory indexes and it should also be possible much 
more easily to create other index orderings, if users find that useful for 
special purposes. (I.e. saving space under conditions where the possible 
queries are known and very restricted.)

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

    $ git pull https://github.com/ajs6f/jena TuplesAreTuples

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

    https://github.com/apache/jena/pull/110.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 #110
    
----
commit 64224ca320fd2989502e7699ec33283dc0fe6ff7
Author: ajs6f <aj...@virginia.edu>
Date:   2015-12-14T18:13:15Z

    Using ColumnMap in transaction in-memory dataset implementation

----


> MInor refactoring in TupleTables
> --------------------------------
>
>                 Key: JENA-1083
>                 URL: https://issues.apache.org/jira/browse/JENA-1083
>             Project: Apache Jena
>          Issue Type: Improvement
>          Components: ARQ
>            Reporter: A. Soroka
>            Priority: Minor
>
> There are some minor refactorings available for TupleTable and its subtypes, 
> particularly PMapTripleTable and PMapQuadTable that will clarify their use. 
> Specifically, current impls of those abstract types have to override several 
> methods for adding, removing, and finding tuples. In fact, the only 
> information being added when those methods are overridden is conversion 
> between canonical and internal tuple ordering. This refactoring is to provide 
> methods that do that conversion and nothing else, which will make two methods 
> the most that any implementation of those abstract classes will have to 
> provide.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to