Aklakan opened a new pull request, #3264:
URL: https://github.com/apache/jena/pull/3264

   As the title says. (If desired I can create an issue). The issue occurred to 
me when I was trying to post process variable names in Ops after the optimizer 
had already run. OpPropFunc arguments were not affected then.
   One detail question is whether the NodeTransform should affect the property 
function URI.
   My argument against it are:
   * Its probably desired to prevent a node transform from mapping a  property 
function name to a blank node or literal.
   * Renaming property function names in the BGPs could be done before the 
optimizer runs.
   * In the work case, it would still possible to subclass TransformCopy just 
for that case.
   
   ----
   
    - [x] Tests are included.
    - [x] Commits have been squashed to remove intermediate development commit 
messages.
    - (Key commit messages start with the issue number (GH-xxxx))
   
   By submitting this pull request, I acknowledge that I am making a 
contribution to the Apache Software Foundation under the terms and conditions 
of the [Contributor's 
Agreement](https://www.apache.org/licenses/contributor-agreements.html).
   
   ----
   
   See the [Apache Jena "Contributing" 
guide](https://github.com/apache/jena/blob/main/CONTRIBUTING.md).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscr...@jena.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@jena.apache.org
For additional commands, e-mail: pr-h...@jena.apache.org

Reply via email to