You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@linkis.apache.org by GitBox <gi...@apache.org> on 2022/07/28 05:32:41 UTC

[GitHub] [incubator-linkis] AaronLinOops opened a new pull request, #2540: [Feature] introduced a new image, called LDH (Linkis's hadoop all-in-one image)

AaronLinOops opened a new pull request, #2540:
URL: https://github.com/apache/incubator-linkis/pull/2540

   ### What is the purpose of the change
   introduced a new image, called LDH (Linkis's hadoop all-in-one image), which provides a pseudo-distributed hadoop cluster for testing quickly.
   
   ### Brief change log
   - a Dockerfile for LDH image
   - some script to bring up ldh cluster easily
   
   ### Verifying this change
   (Please pick either of the following options)  
   This change is a trivial rework / code cleanup without any test coverage.  
   
   ### Does this pull request potentially affect one of the following parts:
   - Dependencies (does it add or upgrade a dependency): (no)
   - Anything that affects deployment: (no)
   - The MGS(Microservice Governance Services), i.e., Spring Cloud Gateway, OpenFeign, Eureka.: (no)
   
   ### Documentation
   - Does this pull request introduce a new feature? (yes)
   - If yes, how is the feature documented? (docs)


-- 
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: notifications-unsubscribe@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


[GitHub] [incubator-linkis] legendtkl merged pull request #2540: [Feature] introduced a new image, called LDH (Linkis's hadoop all-in-one image)

Posted by GitBox <gi...@apache.org>.
legendtkl merged PR #2540:
URL: https://github.com/apache/incubator-linkis/pull/2540


-- 
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: notifications-unsubscribe@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


[GitHub] [incubator-linkis] legendtkl commented on pull request #2540: [Feature] introduced a new image, called LDH (Linkis's hadoop all-in-one image)

Posted by GitBox <gi...@apache.org>.
legendtkl commented on PR #2540:
URL: https://github.com/apache/incubator-linkis/pull/2540#issuecomment-1200365881

   LGTM


-- 
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: notifications-unsubscribe@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


[GitHub] [incubator-linkis] legendtkl commented on a diff in pull request #2540: [Feature] introduced a new image, called LDH (Linkis's hadoop all-in-one image)

Posted by GitBox <gi...@apache.org>.
legendtkl commented on code in PR #2540:
URL: https://github.com/apache/incubator-linkis/pull/2540#discussion_r933813510


##########
linkis-dist/helm/scripts/resources/ldh/configmaps/configmap-flink.yaml:
##########
@@ -0,0 +1,273 @@
+# 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.
+---
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: flink-conf
+data:
+  flink-conf.yaml: |
+    #==============================================================================
+    # Common
+    #==============================================================================
+
+    # The external address of the host on which the JobManager runs and can be
+    # reached by the TaskManagers and any clients which want to connect. This setting
+    # is only used in Standalone mode and may be overwritten on the JobManager side
+    # by specifying the --host <hostname> parameter of the bin/jobmanager.sh executable.
+    # In high availability mode, if you use the bin/start-cluster.sh script and setup
+    # the conf/masters file, this will be taken care of automatically. Yarn
+    # automatically configure the host name based on the hostname of the node where the
+    # JobManager runs.
+
+    jobmanager.rpc.address: localhost
+
+    # The RPC port where the JobManager is reachable.
+
+    jobmanager.rpc.port: 6123
+
+
+    # The total process memory size for the JobManager.
+    #
+    # Note this accounts for all memory usage within the JobManager process, including JVM metaspace and other overhead.
+
+    jobmanager.memory.process.size: 1600m
+
+
+    # The total process memory size for the TaskManager.
+    #
+    # Note this accounts for all memory usage within the TaskManager process, including JVM metaspace and other overhead.
+
+    taskmanager.memory.process.size: 1728m
+
+    # To exclude JVM metaspace and overhead, please, use total Flink memory size instead of 'taskmanager.memory.process.size'.
+    # It is not recommended to set both 'taskmanager.memory.process.size' and Flink memory.
+    #
+    # taskmanager.memory.flink.size: 1280m
+
+    # The number of task slots that each TaskManager offers. Each slot runs one parallel pipeline.
+
+    taskmanager.numberOfTaskSlots: 1
+
+    # The parallelism used for programs that did not specify and other parallelism.
+
+    parallelism.default: 1
+
+    # The default file system scheme and authority.
+    #
+    # By default file paths without scheme are interpreted relative to the local
+    # root file system 'file:///'. Use this to override the default and interpret
+    # relative paths relative to a different file system,
+    # for example 'hdfs://mynamenode:12345'
+    #
+    # fs.default-scheme
+
+    #==============================================================================
+    # High Availability
+    #==============================================================================
+
+    # The high-availability mode. Possible options are 'NONE' or 'zookeeper'.
+    #
+    # high-availability: zookeeper
+
+    # The path where metadata for master recovery is persisted. While ZooKeeper stores
+    # the small ground truth for checkpoint and leader election, this location stores
+    # the larger objects, like persisted dataflow graphs.
+    #
+    # Must be a durable file system that is accessible from all nodes
+    # (like HDFS, S3, Ceph, nfs, ...)
+    #
+    # high-availability.storageDir: hdfs:///flink/ha/
+
+    # The list of ZooKeeper quorum peers that coordinate the high-availability
+    # setup. This must be a list of the form:
+    # "host1:clientPort,host2:clientPort,..." (default clientPort: 2181)
+    #
+    # high-availability.zookeeper.quorum: localhost:2181
+
+
+    # ACL options are based on https://zookeeper.apache.org/doc/r3.1.2/zookeeperProgrammers.html#sc_BuiltinACLSchemes
+    # It can be either "creator" (ZOO_CREATE_ALL_ACL) or "open" (ZOO_OPEN_ACL_UNSAFE)
+    # The default value is "open" and it can be changed to "creator" if ZK security is enabled
+    #
+    # high-availability.zookeeper.client.acl: open
+
+    #==============================================================================
+    # Fault tolerance and checkpointing
+    #==============================================================================
+
+    # The backend that will be used to store operator state checkpoints if
+    # checkpointing is enabled. Checkpointing is enabled when execution.checkpointing.interval > 0.
+    #
+    # Execution checkpointing related parameters. Please refer to CheckpointConfig and ExecutionCheckpointingOptions for more details.
+    #
+    # execution.checkpointing.interval: 3min
+    # execution.checkpointing.externalized-checkpoint-retention: [DELETE_ON_CANCELLATION, RETAIN_ON_CANCELLATION]
+    # execution.checkpointing.max-concurrent-checkpoints: 1
+    # execution.checkpointing.min-pause: 0
+    # execution.checkpointing.mode: [EXACTLY_ONCE, AT_LEAST_ONCE]
+    # execution.checkpointing.timeout: 10min
+    # execution.checkpointing.tolerable-failed-checkpoints: 0
+    # execution.checkpointing.unaligned: false
+    #
+    # Supported backends are 'jobmanager', 'filesystem', 'rocksdb', or the
+    # <class-name-of-factory>.
+    #
+    # state.backend: filesystem
+
+    # Directory for checkpoints filesystem, when using any of the default bundled
+    # state backends.
+    #
+    # state.checkpoints.dir: hdfs://namenode-host:port/flink-checkpoints
+
+    # Default target directory for savepoints, optional.
+    #
+    # state.savepoints.dir: hdfs://namenode-host:port/flink-savepoints
+
+    # Flag to enable/disable incremental checkpoints for backends that
+    # support incremental checkpoints (like the RocksDB state backend).
+    #
+    # state.backend.incremental: false
+
+    # The failover strategy, i.e., how the job computation recovers from task failures.
+    # Only restart tasks that may have been affected by the task failure, which typically includes
+    # downstream tasks and potentially upstream tasks if their produced data is no longer available for consumption.
+
+    jobmanager.execution.failover-strategy: region
+
+    #==============================================================================
+    # Rest & web frontend
+    #==============================================================================
+
+    # The port to which the REST client connects to. If rest.bind-port has
+    # not been specified, then the server will bind to this port as well.
+    #
+    rest.port: 8090
+
+    # The address to which the REST client will connect to
+    #
+    #rest.address: 0.0.0.0
+
+    # Port range for the REST and web server to bind to.
+    #
+    #rest.bind-port: 8090-8100
+
+    # The address that the REST & web server binds to
+    #
+    #rest.bind-address: 0.0.0.0
+
+    # Flag to specify whether job submission is enabled from the web-based
+    # runtime monitor. Uncomment to disable.
+
+    #web.submit.enable: false
+
+    # Flag to specify whether job cancellation is enabled from the web-based
+    # runtime monitor. Uncomment to disable.
+
+    #web.cancel.enable: false
+
+    #==============================================================================
+    # Advanced
+    #==============================================================================
+
+    # Override the directories for temporary files. If not specified, the
+    # system-specific Java temporary directory (java.io.tmpdir property) is taken.
+    #
+    # For framework setups on Yarn, Flink will automatically pick up the
+    # containers' temp directories without any need for configuration.
+    #
+    # Add a delimited list for multiple directories, using the system directory
+    # delimiter (colon ':' on unix) or a comma, e.g.:
+    #     /data1/tmp:/data2/tmp:/data3/tmp
+    #
+    # Note: Each directory entry is read from and written to by a different I/O
+    # thread. You can include the same directory multiple times in order to create
+    # multiple I/O threads against that directory. This is for example relevant for
+    # high-throughput RAIDs.
+    #
+    # io.tmp.dirs: /tmp
+
+    # The classloading resolve order. Possible values are 'child-first' (Flink's default)
+    # and 'parent-first' (Java's default).
+    #
+    # Child first classloading allows users to use different dependency/library
+    # versions in their application than those in the classpath. Switching back
+    # to 'parent-first' may help with debugging dependency issues.
+    #
+    # classloader.resolve-order: child-first
+
+    # The amount of memory going to the network stack. These numbers usually need
+    # no tuning. Adjusting them may be necessary in case of an "Insufficient number
+    # of network buffers" error. The default min is 64MB, the default max is 1GB.
+    #
+    # taskmanager.memory.network.fraction: 0.1
+    # taskmanager.memory.network.min: 64mb
+    # taskmanager.memory.network.max: 1gb
+
+    #==============================================================================
+    # Flink Cluster Security Configuration
+    #==============================================================================
+
+    # Kerberos authentication for various components - Hadoop, ZooKeeper, and connectors -
+    # may be enabled in four steps:
+    # 1. configure the local krb5.conf file
+    # 2. provide Kerberos credentials (either a keytab or a ticket cache w/ kinit)
+    # 3. make the credentials available to various JAAS login contexts
+    # 4. configure the connector to use JAAS/SASL
+
+    # The below configure how Kerberos credentials are provided. A keytab will be used instead of
+    # a ticket cache if the keytab path and principal are set.
+
+    # security.kerberos.login.use-ticket-cache: true
+    # security.kerberos.login.keytab: /path/to/kerberos/keytab
+    # security.kerberos.login.principal: flink-user
+
+    # The configuration below defines which JAAS login contexts
+
+    # security.kerberos.login.contexts: Client,KafkaClient
+
+    #==============================================================================
+    # ZK Security Configuration
+    #==============================================================================
+
+    # Below configurations are applicable if ZK ensemble is configured for security
+
+    # Override below configuration to provide custom ZK service name if configured
+    # zookeeper.sasl.service-name: zookeeper
+
+    # The configuration below must match one of the values set in "security.kerberos.login.contexts"
+    # zookeeper.sasl.login-context-name: Client
+
+    #==============================================================================
+    # HistoryServer
+    #==============================================================================
+
+    # The HistoryServer is started and stopped via bin/historyserver.sh (start|stop)
+
+    # Directory to upload completed jobs to. Add this directory to the list of
+    # monitored directories of the HistoryServer as well (see below).
+    #jobmanager.archive.fs.dir: hdfs:///completed-jobs/
+
+    # The address under which the web-based HistoryServer listens.
+    #historyserver.web.address: 0.0.0.0
+
+    # The port under which the web-based HistoryServer listens.
+    #historyserver.web.port: 8082
+
+    # Comma separated list of directories to monitor for completed jobs.
+    #historyserver.archive.fs.dir: hdfs:///completed-jobs/
+
+    # Interval in milliseconds for refreshing the monitored directories.
+    #historyserver.archive.fs.refresh-interval: 10000

Review Comment:
   empty line is recommanded



-- 
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: notifications-unsubscribe@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


[GitHub] [incubator-linkis] codecov[bot] commented on pull request #2540: [Feature] introduced a new image, called LDH (Linkis's hadoop all-in-one image)

Posted by GitBox <gi...@apache.org>.
codecov[bot] commented on PR #2540:
URL: https://github.com/apache/incubator-linkis/pull/2540#issuecomment-1197731240

   # [Codecov](https://codecov.io/gh/apache/incubator-linkis/pull/2540?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#2540](https://codecov.io/gh/apache/incubator-linkis/pull/2540?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (e4c0643) into [dev-1.3.0](https://codecov.io/gh/apache/incubator-linkis/commit/15f536b671b3a115cd9192c175cb7925ffc7b05f?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (15f536b) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   ```diff
   @@             Coverage Diff              @@
   ##             dev-1.3.0    #2540   +/-   ##
   ============================================
     Coverage        17.84%   17.84%           
     Complexity        1077     1077           
   ============================================
     Files              595      595           
     Lines            17668    17668           
     Branches          2635     2635           
   ============================================
     Hits              3152     3152           
     Misses           14092    14092           
     Partials           424      424           
   ```
   
   
   
   Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
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: notifications-unsubscribe@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org


[GitHub] [incubator-linkis] peacewong commented on a diff in pull request #2540: [Feature] introduced a new image, called LDH (Linkis's hadoop all-in-one image)

Posted by GitBox <gi...@apache.org>.
peacewong commented on code in PR #2540:
URL: https://github.com/apache/incubator-linkis/pull/2540#discussion_r933336985


##########
linkis-dist/docker/ldh.Dockerfile:
##########
@@ -0,0 +1,78 @@
+#
+# 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.
+#
+
+######################################################################
+# hadoop all in one image
+######################################################################
+
+ARG IMAGE_BASE=centos:7
+
+FROM ${IMAGE_BASE} as linkis-ldh
+
+ARG JDK_VERSION=1.8.0-openjdk
+ARG JDK_BUILD_REVISION=1.8.0.332.b09-1.el7_9
+ARG MYSQL_JDBC_VERSION=5.1.49
+
+ARG HADOOP_VERSION=2.7.0

Review Comment:
   It is best to match the version numbers of hive, hadoop, and spark used by linkis by default



-- 
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: notifications-unsubscribe@linkis.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org