You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/02/14 19:40:32 UTC
[GitHub] [camel] marcelloraffaele opened a new pull request #5085: CAMEL-15964: created camel-google-storage component
marcelloraffaele opened a new pull request #5085:
URL: https://github.com/apache/camel/pull/5085
Hi, as promised at the Jira issue https://issues.apache.org/jira/browse/CAMEL-15964, I created the camel-google-storage.
I tried to make it similar to AWS-s3 component and to get the best of camel-google-bigquery. I worked on documentation and tests. For the unit test, I have extended the class https://github.com/googleapis/java-storage-nio/blob/master/google-cloud-nio/src/main/java/com/google/cloud/storage/contrib/nio/testing/FakeStorageRpc.java improving some bucket functionalities that help us to test the component.
----------------------------------------------------------------
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
[GitHub] [camel] oscerd commented on pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #5085:
URL: https://github.com/apache/camel/pull/5085#issuecomment-779646648
It's not critical for merging. It should be already fixed, you need to rebase your branch
----------------------------------------------------------------
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
[GitHub] [camel] marcelloraffaele commented on pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
marcelloraffaele commented on pull request #5085:
URL: https://github.com/apache/camel/pull/5085#issuecomment-779644392
Hi, the build is failing cause: "project camel-http: An error has occurred in Checkstyle report generation.: Failed during checkstyle execution". Can I do something to fix 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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] oscerd commented on pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #5085:
URL: https://github.com/apache/camel/pull/5085#issuecomment-779376386
Yeah, no problem. Just code formatting.
----------------------------------------------------------------
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
[GitHub] [camel] oscerd commented on pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #5085:
URL: https://github.com/apache/camel/pull/5085#issuecomment-780008470
Looks good, I'll try to merge this week. Thanks.
----------------------------------------------------------------
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
[GitHub] [camel] oscerd commented on a change in pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
oscerd commented on a change in pull request #5085:
URL: https://github.com/apache/camel/pull/5085#discussion_r576150975
##########
File path: components/camel-google-storage/src/main/java/org/apache/camel/component/google/storage/GoogleCloudStorageComponentConfiguration.java
##########
@@ -0,0 +1,239 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.google.storage;
+
+import com.google.cloud.storage.Storage;
+import com.google.cloud.storage.StorageClass;
+import org.apache.camel.RuntimeCamelException;
+import org.apache.camel.spi.Metadata;
+import org.apache.camel.spi.UriParam;
+import org.apache.camel.spi.UriParams;
+import org.apache.camel.spi.UriPath;
+
+@UriParams
+public class GoogleCloudStorageComponentConfiguration implements Cloneable {
+
+ @UriPath(label = "common", description = "Bucket name or ARN")
+ @Metadata(required = true)
+ private String bucketName;
+
+ @UriParam(label = "common", description = "Service account key to authenticate an application as a service account")
+ private String serviceAccountKey;
+
+ @UriParam(label = "producer",
+ enums = "copyObject,listObjects,deleteObject,deleteBucket,listBuckets,getObject,createDownloadLink")
+ private GoogleCloudStorageComponentOperations operation;
+
+ @UriParam(label = "producer", description = "The Object name inside the bucket")
+ private String objectName;
+
+ @UriParam(label = "common", defaultValue = "US-EAST1",
+ description = "The Cloud Storage location to use when creating the new buckets")
+ private String storageLocation = "US-EAST1";
+
+ @UriParam(label = "common", defaultValue = "STANDARD",
+ description = "The Cloud Storage class to use when creating the new buckets")
+ private StorageClass storageClass = StorageClass.STANDARD;
+
+ @UriParam(label = "common", defaultValue = "true")
+ private boolean autoCreateBucket = true;
+
+ @UriParam(label = "consumer")
+ private boolean moveAfterRead;
+
+ @UriParam(label = "consumer")
+ private String destinationBucket;
+
+ @UriParam(label = "consumer", defaultValue = "true")
+ private boolean deleteAfterRead = true;
+
+ @UriParam(label = "consumer", defaultValue = "true")
+ private boolean includeBody = true;
+
+ @UriParam(label = "consumer", defaultValue = "true")
+ private boolean includeFolders = true;
+
+ @UriParam
+ private Storage storageClient;
Review comment:
This should be autowired (one of the possible metadata of uriparam), in this way if we have a storage client instance in the registry it will be used ootb.
##########
File path: components/camel-google-storage/src/generated/resources/google-storage.json
##########
@@ -0,0 +1,14 @@
+{
+ "other": {
+ "kind": "other",
+ "name": "google-storage",
+ "title": "Google Storage",
+ "description": "Camel Component for Google Cloud Platform Storage",
+ "deprecated": false,
+ "firstVersion": "3.8.0",
Review comment:
This will be re-generated again once firstVersion will be 3.9.0
##########
File path: components/camel-google-storage/pom.xml
##########
@@ -0,0 +1,126 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements. See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+
+-->
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
+
+ <modelVersion>4.0.0</modelVersion>
+
+ <parent>
+ <groupId>org.apache.camel</groupId>
+ <artifactId>components</artifactId>
+ <version>3.9.0-SNAPSHOT</version>
+ </parent>
+
+ <artifactId>camel-google-storage</artifactId>
+ <packaging>jar</packaging>
+ <name>Camel :: Google Storage</name>
+ <description>Camel Component for Google Cloud Platform Storage</description>
+
+ <properties>
+ <firstVersion>3.8.0</firstVersion>
Review comment:
This should be 3.9.0
----------------------------------------------------------------
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
[GitHub] [camel] omarsmak commented on a change in pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
omarsmak commented on a change in pull request #5085:
URL: https://github.com/apache/camel/pull/5085#discussion_r576903243
##########
File path: components/camel-google-storage/src/main/java/org/apache/camel/component/google/storage/GoogleCloudStorageConsumer.java
##########
@@ -0,0 +1,260 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.google.storage;
+
+import java.util.LinkedList;
+import java.util.List;
+import java.util.Queue;
+
+import com.google.cloud.storage.Blob;
+import com.google.cloud.storage.BlobId;
+import com.google.cloud.storage.Bucket;
+import com.google.cloud.storage.CopyWriter;
+import com.google.cloud.storage.Storage;
+import com.google.cloud.storage.Storage.CopyRequest;
+import org.apache.camel.AsyncCallback;
+import org.apache.camel.Exchange;
+import org.apache.camel.ExtendedExchange;
+import org.apache.camel.Processor;
+import org.apache.camel.spi.Synchronization;
+import org.apache.camel.support.ScheduledBatchPollingConsumer;
+import org.apache.camel.util.CastUtils;
+import org.apache.camel.util.ObjectHelper;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+public class GoogleCloudStorageConsumer extends ScheduledBatchPollingConsumer {
+
+ private static final Logger LOG = LoggerFactory.getLogger(GoogleCloudStorageConsumer.class);
+
+ //private String marker;
Review comment:
can this be removed?
##########
File path: components/camel-google-storage/src/main/java/org/apache/camel/component/google/storage/GoogleCloudStorageConsumer.java
##########
@@ -0,0 +1,260 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.google.storage;
+
+import java.util.LinkedList;
+import java.util.List;
+import java.util.Queue;
+
+import com.google.cloud.storage.Blob;
+import com.google.cloud.storage.BlobId;
+import com.google.cloud.storage.Bucket;
+import com.google.cloud.storage.CopyWriter;
+import com.google.cloud.storage.Storage;
+import com.google.cloud.storage.Storage.CopyRequest;
+import org.apache.camel.AsyncCallback;
+import org.apache.camel.Exchange;
+import org.apache.camel.ExtendedExchange;
+import org.apache.camel.Processor;
+import org.apache.camel.spi.Synchronization;
+import org.apache.camel.support.ScheduledBatchPollingConsumer;
+import org.apache.camel.util.CastUtils;
+import org.apache.camel.util.ObjectHelper;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+public class GoogleCloudStorageConsumer extends ScheduledBatchPollingConsumer {
+
+ private static final Logger LOG = LoggerFactory.getLogger(GoogleCloudStorageConsumer.class);
+
+ //private String marker;
+ //private transient String consumerToString;
+
+ public GoogleCloudStorageConsumer(GoogleCloudStorageEndpoint endpoint, Processor processor) {
+ super(endpoint, processor);
+ }
+
+ @Override
+ protected void doStart() throws Exception {
+ super.doStart();
+
+ if (getConfiguration().isMoveAfterRead()) {
+
+ Bucket bucket = getStorageClient().get(getConfiguration().getDestinationBucket());
+ if (bucket != null) {
+ LOG.trace("Bucket [{}] already exists", bucket.getName());
+ return;
+ } else {
+ LOG.trace("Destination Bucket [{}] doesn't exist yet", getConfiguration().getDestinationBucket());
+ if (getConfiguration().isAutoCreateBucket()) {
+ // creates the new bucket because it doesn't exist yet
+ GoogleCloudStorageEndpoint.createNewBucket(getConfiguration().getDestinationBucket(), getConfiguration(),
+ getStorageClient());
+ }
+ }
+
+ }
+ }
+
+ @Override
+ protected int poll() throws Exception {
+ // must reset for each poll
+ shutdownRunningTask = null;
+ pendingExchanges = 0;
+
+ String fileName = getConfiguration().getObjectName();
+ String bucketName = getConfiguration().getBucketName();
+ Queue<Exchange> exchanges = new LinkedList<>();
+
+ if (fileName != null) {
+ LOG.trace("Getting object in bucket [{}] with file name [{}]...", bucketName, fileName);
+
+ Blob blob = getStorageClient().get(bucketName, fileName);
+
+ exchanges = createExchanges(blob, fileName);
+ } else {
+ LOG.trace("Queueing objects in bucket [{}]...", bucketName);
+
+ List<Blob> bloblist = new LinkedList<>();
+ for (Blob blob : getStorageClient().list(bucketName).iterateAll()) {
+ bloblist.add(blob);
+ }
+
+ if (LOG.isTraceEnabled()) {
+ LOG.trace("Found {} objects in bucket [{}]...", bloblist.size(), bucketName);
+ }
+
+ exchanges = createExchanges(bloblist);
+ }
+
+ return processBatch(CastUtils.cast(exchanges));
+ }
+
+ protected Queue<Exchange> createExchanges(Blob blob, String key) {
+ Queue<Exchange> answer = new LinkedList<>();
+ Exchange exchange = getEndpoint().createExchange(blob, key);
+ answer.add(exchange);
+ return answer;
+ }
+
+ protected Queue<Exchange> createExchanges(List<Blob> blobList) {
+ if (LOG.isTraceEnabled()) {
+ LOG.trace("Received {} messages in this poll", blobList.size());
+ }
+
+ Queue<Exchange> answer = new LinkedList<>();
+ try {
+ for (Blob blob : blobList) {
+
+ if (includeObject(blob)) {
+ Exchange exchange = getEndpoint().createExchange(blob, blob.getBlobId().getName());
+ answer.add(exchange);
+ }
+
+ }
+ } catch (Exception e) {
+ LOG.warn("Error getting object due: {}", e.getMessage(), e);
+ throw e;
+ }
+
+ return answer;
+ }
+
+ /**
+ * Decide whether to include the Objects in the results
+ *
+ * @param Object
+ * @return true to include, false to exclude
+ */
+ protected boolean includeObject(Blob blob) {
+
+ if (getConfiguration().isIncludeFolders()) {
+ return true;
+ }
+ // Config says to ignore folders/directories
+ return blob.getName().endsWith("/");
+ }
+
+ @Override
+ public int processBatch(Queue<Object> exchanges) throws Exception {
+ int total = exchanges.size();
+
+ for (int index = 0; index < total && isBatchAllowed(); index++) {
+ // only loop if we are started (allowed to run)
+ final Exchange exchange = ObjectHelper.cast(Exchange.class, exchanges.poll());
+ // add current index and total as properties
+ exchange.setProperty(Exchange.BATCH_INDEX, index);
+ exchange.setProperty(Exchange.BATCH_SIZE, total);
+ exchange.setProperty(Exchange.BATCH_COMPLETE, index == total - 1);
+
+ // update pending number of exchanges
+ pendingExchanges = total - index - 1;
+
+ // add on completion to handle after work when the exchange is done
+ exchange.adapt(ExtendedExchange.class).addOnCompletion(new Synchronization() {
+ public void onComplete(Exchange exchange) {
+ processCommit(exchange);
+ }
+
+ public void onFailure(Exchange exchange) {
+ processRollback(exchange);
+ }
+
+ @Override
+ public String toString() {
+ return "ConsumerOnCompletion";
+ }
+ });
+
+ LOG.trace("Processing exchange [{}]...", exchange);
+ getAsyncProcessor().process(exchange, new AsyncCallback() {
+ @Override
+ public void done(boolean doneSync) {
+ LOG.trace("Processing exchange [{}] done.", exchange);
+ }
+ });
+ }
+
+ return total;
+ }
+
+ /**
+ * Strategy to delete the message after being processed.
+ *
+ * @param exchange the exchange
+ */
+ protected void processCommit(Exchange exchange) {
+ //LOG.info("processCommit");
Review comment:
unused comment
----------------------------------------------------------------
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
[GitHub] [camel] marcelloraffaele commented on a change in pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
marcelloraffaele commented on a change in pull request #5085:
URL: https://github.com/apache/camel/pull/5085#discussion_r576344722
##########
File path: components/camel-google-storage/src/generated/resources/google-storage.json
##########
@@ -0,0 +1,14 @@
+{
+ "other": {
+ "kind": "other",
+ "name": "google-storage",
+ "title": "Google Storage",
+ "description": "Camel Component for Google Cloud Platform Storage",
+ "deprecated": false,
+ "firstVersion": "3.8.0",
Review comment:
done
##########
File path: components/camel-google-storage/src/main/java/org/apache/camel/component/google/storage/GoogleCloudStorageComponentConfiguration.java
##########
@@ -0,0 +1,239 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.camel.component.google.storage;
+
+import com.google.cloud.storage.Storage;
+import com.google.cloud.storage.StorageClass;
+import org.apache.camel.RuntimeCamelException;
+import org.apache.camel.spi.Metadata;
+import org.apache.camel.spi.UriParam;
+import org.apache.camel.spi.UriParams;
+import org.apache.camel.spi.UriPath;
+
+@UriParams
+public class GoogleCloudStorageComponentConfiguration implements Cloneable {
+
+ @UriPath(label = "common", description = "Bucket name or ARN")
+ @Metadata(required = true)
+ private String bucketName;
+
+ @UriParam(label = "common", description = "Service account key to authenticate an application as a service account")
+ private String serviceAccountKey;
+
+ @UriParam(label = "producer",
+ enums = "copyObject,listObjects,deleteObject,deleteBucket,listBuckets,getObject,createDownloadLink")
+ private GoogleCloudStorageComponentOperations operation;
+
+ @UriParam(label = "producer", description = "The Object name inside the bucket")
+ private String objectName;
+
+ @UriParam(label = "common", defaultValue = "US-EAST1",
+ description = "The Cloud Storage location to use when creating the new buckets")
+ private String storageLocation = "US-EAST1";
+
+ @UriParam(label = "common", defaultValue = "STANDARD",
+ description = "The Cloud Storage class to use when creating the new buckets")
+ private StorageClass storageClass = StorageClass.STANDARD;
+
+ @UriParam(label = "common", defaultValue = "true")
+ private boolean autoCreateBucket = true;
+
+ @UriParam(label = "consumer")
+ private boolean moveAfterRead;
+
+ @UriParam(label = "consumer")
+ private String destinationBucket;
+
+ @UriParam(label = "consumer", defaultValue = "true")
+ private boolean deleteAfterRead = true;
+
+ @UriParam(label = "consumer", defaultValue = "true")
+ private boolean includeBody = true;
+
+ @UriParam(label = "consumer", defaultValue = "true")
+ private boolean includeFolders = true;
+
+ @UriParam
+ private Storage storageClient;
Review comment:
done
----------------------------------------------------------------
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
[GitHub] [camel] mathieu-souchet commented on pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
mathieu-souchet commented on pull request #5085:
URL: https://github.com/apache/camel/pull/5085#issuecomment-785802733
Oh great an official google-cloud-storage component available soon ! 🤩
Past few weeks I did my own implementation for my needs. I've just read your changes, I think it's great but I have one remark :
What about concurrent access to blobs ? In my tests with multiple instances consuming the same file (2 kubernetes pods) I saw concurrent attempts to consume and I fixed that with a BlobExclusiveReadLockStrategy (inspired by the implementation of MarkerFileExclusiveReadLockStrategy to camel-file component)
----------------------------------------------------------------
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
[GitHub] [camel] oscerd commented on pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
oscerd commented on pull request #5085:
URL: https://github.com/apache/camel/pull/5085#issuecomment-780334330
Thanks, this has been merged on master.
----------------------------------------------------------------
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
[GitHub] [camel] marcelloraffaele commented on pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
marcelloraffaele commented on pull request #5085:
URL: https://github.com/apache/camel/pull/5085#issuecomment-779374954
I made the review but the build now is failing on "camel-http". Nitece that not only this build have this problem.
----------------------------------------------------------------
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
[GitHub] [camel] oscerd closed pull request #5085: CAMEL-15964: created camel-google-storage component
Posted by GitBox <gi...@apache.org>.
oscerd closed pull request #5085:
URL: https://github.com/apache/camel/pull/5085
----------------------------------------------------------------
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