clotton commented on a change in pull request #204:
URL: 
https://github.com/apache/jackrabbit-filevault/pull/204#discussion_r806239844



##########
File path: 
vault-core/src/main/java/org/apache/jackrabbit/vault/util/RepositoryCopier.java
##########
@@ -410,11 +410,17 @@ private void copy(AutoSave autoSave, Node src, Node 
dstParent, String dstName, b
                             currentSize+=s;
                         }
                     } else {
-                        Value v = p.getValue();
-                        dst.setProperty(pName, v);
-                        long s= p.getLength();
-                        totalSize+=s;
-                        currentSize+=s;
+                        // Filter out unwanted properties.  Remove them from 
the destination if they are excluded by filter

Review comment:
       @kwin it's not clear to me how we would intercept the ContentHandler 
found in syscopy, which is an ImportHandler defined in oak-run.   A new 
implementation would, perhaps, need to extend ImportHandler to apply filtering, 
and the pass it on to the exportSystemView call as done already.
   
   If you have any guidance, or examples, that would be greatly appreciated.  
If you'd like to schedule some time to discuss this, I'd welcome it.  




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to