You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2020/01/11 09:11:47 UTC

[GitHub] [flink] xintongsong opened a new pull request #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

xintongsong opened a new pull request #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834
 
 
   ## What is the purpose of the change
   
   This PR replaces `taskmanager.memory.process.size` with `taskmanager.memory.flink.size` for the default `flink-conf.yaml`.
   
   It also resolves several yarn deployment problems of using  `taskmanager.memory.flink.size`.
   
   ## Brief change log
   
   - 1c542f583070354ec7eed451653ba86fa14561f1: Disable max vcore validation test cases.
     - These test cases should have failed already, but is covered by cluster specification memory validation failures.
     - This PR removes the cluster specification memory validation and uncovers the problems.
     - We will create separate issues to track the disabled test cases.
   - 642fcbe8fe13b14c8da0e9a5acf1d5c99c617314: Remove `ClusterSpecification` memory validation.
     - The validation is already covered by generating `TaskExecutorResourceSpec` in `AbstractContainerizedClusterClientFactory`.
     - `ClusterSpecification` stores TM memory in MB. The accuracy loss may cause false negative validation failures, especially when process memory is derived from flink memory and is not a integer value of MB.
   - 96f3e7956b3ac28c96afed20e45bca34cd7ea56e: Hotfix for removing unused env keys.
   - 294dae71fc791d88801810d178aef13cbdb02649: Hotfix for removing unused `numberOfTaskManagers`.
   - 14a36328ad3a42d8a15a4797baf7b33107f16160: Hotfix for removing unnecessary overwriting configuration with `ClusterSpecification`.
     - The `ClusterSpecification` is generated from configuration at the first place.
   - 1edb31fe2aa2466c3cae6119c799be03bf30ed1f: Unset configured flink size if process size is specified via client options `-tm`.
   - 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75: Replace process memory with flink memory in default `flink-conf.yaml`.
   
   ## Verifying this change
   
   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)
     - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: (no)
     - The serializers: (no)
     - The runtime per-record code paths (performance sensitive): (no)
     - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes)
     - The S3 file system connector: (no)
   
   ## Documentation
   
     - Does this pull request introduce a new feature? (no)
     - If yes, how is the feature documented? (not applicable)
   

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:PENDING URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:FAILURE URL:https://travis-ci.com/flink-ci/flink/builds/144305867 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:7b4d3b68911cdfa74557fddf01fa481e68d2e5db Status:PENDING URL:https://travis-ci.com/flink-ci/flink/builds/144310929 TriggerType:PUSH TriggerID:7b4d3b68911cdfa74557fddf01fa481e68d2e5db
   Hash:7b4d3b68911cdfa74557fddf01fa481e68d2e5db Status:PENDING URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4329 TriggerType:PUSH TriggerID:7b4d3b68911cdfa74557fddf01fa481e68d2e5db
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [SUCCESS](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   * 2cc46266d918c400c86789791721e6dd98758954 Travis: [FAILURE](https://travis-ci.com/flink-ci/flink/builds/144305867) Azure: [PENDING](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327) 
   * 7b4d3b68911cdfa74557fddf01fa481e68d2e5db Travis: [PENDING](https://travis-ci.com/flink-ci/flink/builds/144310929) Azure: [PENDING](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4329) 
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:UNKNOWN URL:TBD TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:FAILURE URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:FAILURE URL:https://travis-ci.com/flink-ci/flink/builds/144305867 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:7b4d3b68911cdfa74557fddf01fa481e68d2e5db Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/144310929 TriggerType:PUSH TriggerID:7b4d3b68911cdfa74557fddf01fa481e68d2e5db
   Hash:7b4d3b68911cdfa74557fddf01fa481e68d2e5db Status:SUCCESS URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4329 TriggerType:PUSH TriggerID:7b4d3b68911cdfa74557fddf01fa481e68d2e5db
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [SUCCESS](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   * 2cc46266d918c400c86789791721e6dd98758954 Travis: [FAILURE](https://travis-ci.com/flink-ci/flink/builds/144305867) Azure: [FAILURE](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327) 
   * 7b4d3b68911cdfa74557fddf01fa481e68d2e5db Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/144310929) Azure: [SUCCESS](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4329) 
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] xintongsong closed pull request #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
xintongsong closed pull request #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834
 
 
   

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:PENDING URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [PENDING](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:PENDING URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:PENDING URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [PENDING](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [PENDING](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [SUCCESS](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:UNKNOWN URL:TBD TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [SUCCESS](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   * 2cc46266d918c400c86789791721e6dd98758954 UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] xintongsong commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
xintongsong commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-574587558
 
 
   According to the discussion in the [ML thread](http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Some-feedback-after-trying-out-the-new-FLIP-49-memory-configurations-tc36129.html), we decided not to do this change.

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573298663
 
 
   Thanks a lot for your contribution to the Apache Flink project. I'm the @flinkbot. I help the community
   to review your pull request. We will use this comment to track the progress of the review.
   
   
   ## Automated Checks
   Last check on commit 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 (Sat Jan 11 09:15:14 UTC 2020)
   
   **Warnings:**
    * No documentation files were touched! Remember to keep the Flink docs up to date!
   
   
   <sub>Mention the bot in a comment to re-run the automated checks.</sub>
   ## Review Progress
   
   * ❓ 1. The [description] looks good.
   * ❓ 2. There is [consensus] that the contribution should go into to Flink.
   * ❓ 3. Needs [attention] from.
   * ❓ 4. The change fits into the overall [architecture].
   * ❓ 5. Overall code [quality] is good.
   
   Please see the [Pull Request Review Guide](https://flink.apache.org/contributing/reviewing-prs.html) for a full explanation of the review process.<details>
    The Bot is tracking the review progress through labels. Labels are applied according to the order of the review items. For consensus, approval by a Flink committer of PMC member is required <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot approve description` to approve one or more aspects (aspects: `description`, `consensus`, `architecture` and `quality`)
    - `@flinkbot approve all` to approve all aspects
    - `@flinkbot approve-until architecture` to approve everything until `architecture`
    - `@flinkbot attention @username1 [@username2 ..]` to require somebody's attention
    - `@flinkbot disapprove architecture` to remove an approval you gave earlier
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:PENDING URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:FAILURE URL:https://travis-ci.com/flink-ci/flink/builds/144305867 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:7b4d3b68911cdfa74557fddf01fa481e68d2e5db Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/144310929 TriggerType:PUSH TriggerID:7b4d3b68911cdfa74557fddf01fa481e68d2e5db
   Hash:7b4d3b68911cdfa74557fddf01fa481e68d2e5db Status:PENDING URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4329 TriggerType:PUSH TriggerID:7b4d3b68911cdfa74557fddf01fa481e68d2e5db
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [SUCCESS](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   * 2cc46266d918c400c86789791721e6dd98758954 Travis: [FAILURE](https://travis-ci.com/flink-ci/flink/builds/144305867) Azure: [PENDING](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327) 
   * 7b4d3b68911cdfa74557fddf01fa481e68d2e5db Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/144310929) Azure: [PENDING](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4329) 
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services

[GitHub] [flink] xintongsong commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
xintongsong commented on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573298450
 
 
   cc @azagrebin 

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


With regards,
Apache Git Services

[GitHub] [flink] flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml

Posted by GitBox <gi...@apache.org>.
flinkbot edited a comment on issue #10834: [FLINK-15530][dist] Replace process memory with flink memory for TMs in default flink-conf.yaml
URL: https://github.com/apache/flink/pull/10834#issuecomment-573300994
 
 
   <!--
   Meta data
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://travis-ci.com/flink-ci/flink/builds/143993841 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Status:SUCCESS URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266 TriggerType:PUSH TriggerID:78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:PENDING URL:https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:2cc46266d918c400c86789791721e6dd98758954 Status:PENDING URL:https://travis-ci.com/flink-ci/flink/builds/144305867 TriggerType:PUSH TriggerID:2cc46266d918c400c86789791721e6dd98758954
   Hash:7b4d3b68911cdfa74557fddf01fa481e68d2e5db Status:UNKNOWN URL:TBD TriggerType:PUSH TriggerID:7b4d3b68911cdfa74557fddf01fa481e68d2e5db
   -->
   ## CI report:
   
   * 78ce3aaf9f756bb27b20e1f51ff57d0ff4143b75 Travis: [SUCCESS](https://travis-ci.com/flink-ci/flink/builds/143993841) Azure: [SUCCESS](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4266) 
   * 2cc46266d918c400c86789791721e6dd98758954 Travis: [PENDING](https://travis-ci.com/flink-ci/flink/builds/144305867) Azure: [PENDING](https://dev.azure.com/rmetzger/5bd3ef0a-4359-41af-abca-811b04098d2e/_build/results?buildId=4327) 
   * 7b4d3b68911cdfa74557fddf01fa481e68d2e5db UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run travis` re-run the last Travis build
    - `@flinkbot run azure` re-run the last Azure build
   </details>

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


With regards,
Apache Git Services