[
https://issues.apache.org/jira/browse/BEAM-6058?focusedWorklogId=166192&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-166192
]
ASF GitHub Bot logged work on BEAM-6058:
----------------------------------------
Author: ASF GitHub Bot
Created on: 14/Nov/18 23:52
Start Date: 14/Nov/18 23:52
Worklog Time Spent: 10m
Work Description: swegner commented on a change in pull request #7042:
[BEAM-6058] Publish gRPC 1.13.1 as a vendored dependency.
URL: https://github.com/apache/beam/pull/7042#discussion_r233659725
##########
File path: buildSrc/src/main/groovy/org/apache/beam/gradle/GrpcVendoring.groovy
##########
@@ -69,40 +73,69 @@ class GrpcVendoring {
// ensures that there are no classes outside of the 'org.apache.beam'
namespace.
String prefix = "org.apache.beam.vendor.grpc.v1_13_1";
- return {
+ return [
// guava uses the com.google.common and com.google.thirdparty package
namespaces
- relocate "com.google.common", "${prefix}.com.google.common"
- relocate "com.google.thirdparty", "${prefix}.com.google.thirdparty"
+ "com.google.common": "${prefix}.com.google.common",
+ "com.google.thirdparty": "${prefix}.com.google.thirdparty",
- relocate "com.google.protobuf", "${prefix}.com.google.protobuf"
- relocate "com.google.gson", "${prefix}.com.google.gson"
- relocate "io.grpc", "${prefix}.io.grpc"
- relocate "com.google.auth", "${prefix}.com.google.auth"
- relocate "com.google.api", "${prefix}.com.google.api"
- relocate "com.google.cloud", "${prefix}.com.google.cloud"
- relocate "com.google.logging", "${prefix}.com.google.logging"
- relocate "com.google.longrunning", "${prefix}.com.google.longrunning"
- relocate "com.google.rpc", "${prefix}.com.google.rpc"
- relocate "com.google.type", "${prefix}.com.google.type"
- relocate "io.opencensus", "${prefix}.io.opencensus"
+ "com.google.protobuf": "${prefix}.com.google.protobuf",
Review comment:
nit: these all seem to follow the same convention: `"original.package":
"${prefix}.original.package",` It would be slightly less verbose to maintain
just a list of `original.package`'s and generate the map from it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 166192)
Time Spent: 3h 40m (was: 3.5h)
> Support flink config directory for flink runner.
> ------------------------------------------------
>
> Key: BEAM-6058
> URL: https://issues.apache.org/jira/browse/BEAM-6058
> Project: Beam
> Issue Type: Bug
> Components: runner-flink
> Reporter: Ankur Goenka
> Assignee: Ankur Goenka
> Priority: Major
> Time Spent: 3h 40m
> Remaining Estimate: 0h
>
> Give user option to use flink configuration from a config directory.
> Earlier we only used jars to submit flink jobs which by default picks the
> cluster flink properties.
> Portability uses remote environment to submit flink job. In Remote
> execution, flink uses the client config instead of the cluster config. This
> require us to submit the custom flink properties via flink client.
> Suggestion is to read the flink config from a flink configuration folder and
> set them to the environment.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
