[GitHub] jena pull request #314: JENA-1430

[afs]

Github user afs commented on a diff in the pull request:

    https://github.com/apache/jena/pull/314#discussion_r153754571
  
    --- Diff: 
jena-arq/src/main/java/org/apache/jena/sparql/core/assembler/DatasetAssemblerVocab.java
 ---
    @@ -31,7 +31,7 @@
         // General dataset
         public static final Resource tDataset            = 
ResourceFactory.createResource(NS+"RDFDataset") ;
         // In-memory dataset
    -    public static final Resource tDatasetTxnMem      = 
ResourceFactory.createResource(NS+"DatasetTxnMem") ;
    +    public static final Resource tMemoryDataset      = 
ResourceFactory.createResource(NS+"MemoryDataset") ;
    --- End diff --
    
    @ajs6f correctly pointed out that `DatasetTxnMem` was in the documentation 
so migration support would be good such as register `InMemDatasetAssembler` 
under both names and log a warning if called as `ja:DatasetTxnMem`
    
    (And text for the release notes)


---