From commits-return-75898-archive-asf-public=cust-asf.ponee.io@beam.apache.org Mon Jun 4 18:40:05 2018 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 14368180636 for ; Mon, 4 Jun 2018 18:40:04 +0200 (CEST) Received: (qmail 4938 invoked by uid 500); 4 Jun 2018 16:40:04 -0000 Mailing-List: contact commits-help@beam.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@beam.apache.org Delivered-To: mailing list commits@beam.apache.org Received: (qmail 4928 invoked by uid 99); 4 Jun 2018 16:40:04 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 04 Jun 2018 16:40:04 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id B4DF81806B9 for ; Mon, 4 Jun 2018 16:40:03 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -109.511 X-Spam-Level: X-Spam-Status: No, score=-109.511 tagged_above=-999 required=6.31 tests=[ENV_AND_HDR_SPF_MATCH=-0.5, KAM_ASCII_DIVIDERS=0.8, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, USER_IN_DEF_SPF_WL=-7.5, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-us.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id G6vQNcgF2R9T for ; Mon, 4 Jun 2018 16:40:02 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-us.apache.org (ASF Mail Server at mx1-lw-us.apache.org) with ESMTP id F117C5F288 for ; Mon, 4 Jun 2018 16:40:01 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id EA43AE0F3E for ; Mon, 4 Jun 2018 16:40:00 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id 50CF6210A0 for ; Mon, 4 Jun 2018 16:40:00 +0000 (UTC) Date: Mon, 4 Jun 2018 16:40:00 +0000 (UTC) From: "ASF GitHub Bot (JIRA)" To: commits@beam.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Work logged] (BEAM-4326) Enforce ErrorProne analysis in the fn-execution project MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/BEAM-4326?focusedWorklogId=108656&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-108656 ] ASF GitHub Bot logged work on BEAM-4326: ---------------------------------------- Author: ASF GitHub Bot Created on: 04/Jun/18 16:39 Start Date: 04/Jun/18 16:39 Worklog Time Spent: 10m Work Description: swegner commented on a change in pull request #5542: [BEAM-4326] Enforce ErrorProne analysis in the fn-execution project URL: https://github.com/apache/beam/pull/5542#discussion_r192806084 ########## File path: sdks/java/fn-execution/src/test/java/org/apache/beam/sdk/fn/test/TestExecutorsTest.java ########## @@ -115,6 +116,7 @@ public void evaluate() throws Throwable { } @Test + @SuppressWarnings("FutureReturnValueIgnored") Review comment: Can you add a brief comment describing why this suppression is safe? ---------------------------------------------------------------- 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: users@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 108656) Time Spent: 40m (was: 0.5h) > Enforce ErrorProne analysis in the fn-execution project > ------------------------------------------------------- > > Key: BEAM-4326 > URL: https://issues.apache.org/jira/browse/BEAM-4326 > Project: Beam > Issue Type: Improvement > Components: sdk-java-harness > Reporter: Scott Wegner > Assignee: Cade Markegard > Priority: Minor > Labels: errorprone, starter > Time Spent: 40m > Remaining Estimate: 0h > > Java ErrorProne static analysis was [recently enabled|https://github.com/apache/beam/pull/5161] in the Gradle build process, but only as warnings. ErrorProne errors are generally useful and easy to fix. Some work was done to [make sdks-java-core ErrorProne-clean|https://github.com/apache/beam/pull/5319] and add enforcement. This task is clean ErrorProne warnings and add enforcement in {{beam-sdks-java-fn-execution}}. Additional context discussed on the [dev list|https://lists.apache.org/thread.html/95aae2785c3cd728c2d3378cbdff2a7ba19caffcd4faa2049d2e2f46@%3Cdev.beam.apache.org%3E]. > Fixing this issue will involve: > # Follow instructions in the [Contribution Guide|https://beam.apache.org/contribute/] to set up a {{beam}} development environment. > # Run the following command to compile and run ErrorProne analysis on the project: {{./gradlew :beam-sdks-java-fn-execution:assemble}} > # Fix each ErrorProne warning from the {{sdks/java/fn-execution}} project. > # In {{sdks/java/fn-execution/build.gradle}}, add {{failOnWarning: true}} to the call the {{applyJavaNature()}} ([example|https://github.com/apache/beam/pull/5319/files#diff-9390c20635aed5f42f83b97506a87333R20]). > This starter issue is sponsored by [~swegner]. Feel free to [reach out|https://beam.apache.org/community/contact-us/] with questions or code review: > * JIRA: [~swegner] > * GitHub: [@swegner|https://github.com/swegner] > * Slack: [@Scott Wegner|https://s.apache.org/beam-slack-channel] > * Email: swegner at google dot com -- This message was sent by Atlassian JIRA (v7.6.3#76005)