[GitHub] [camel] davsclaus commented on a change in pull request #4514: [CAMEL-15757] Add rawPayload support in camel-salesforce composite

[GitBox <gi...@apache.org>]

davsclaus commented on a change in pull request #4514:
URL: https://github.com/apache/camel/pull/4514#discussion_r538481700



##########
File path: components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/client/DefaultCompositeApiClient.java
##########
@@ -93,6 +93,35 @@ public DefaultCompositeApiClient(final SalesforceEndpointConfig configuration, f
         xStreamCompositeTree.alias("SObjectTreeResponse", SObjectTreeResponse.class);
     }
 
+    public void submitCompositeRaw(
+            final InputStream raw, final Map<String, List<String>> headers,
+            final ResponseCallback<InputStream> callback,
+            final String sObjectName, final String extId, String compositeMethod)
+            throws SalesforceException {
+        checkCompositeFormat(format, SObjectComposite.REQUIRED_PAYLOAD_FORMAT);
+
+        final String url = String.format("%s%s/%s/%s/%s", versionUrl(), "composite", "sobjects", sObjectName, extId);
+
+        System.out.println("URL: " + url);

Review comment:
       Remove system out, or change into a debug LOG

##########
File path: components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/SalesforceEndpointConfig.java
##########
@@ -126,6 +127,8 @@
     @UriParam
     private String apexMethod;
     @UriParam
+    private String compositeMethod;

Review comment:
       Can this be used both on consumer or producer? If not then add label to mark it as consumer or producer only.




----------------------------------------------------------------
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.

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