From commits-return-26463-archive-asf-public=cust-asf.ponee.io@airflow.incubator.apache.org Mon Oct 29 15:09:44 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 47701180677 for ; Mon, 29 Oct 2018 15:09:44 +0100 (CET) Received: (qmail 23277 invoked by uid 500); 29 Oct 2018 14:09:43 -0000 Mailing-List: contact commits-help@airflow.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@airflow.incubator.apache.org Delivered-To: mailing list commits@airflow.incubator.apache.org Received: (qmail 23268 invoked by uid 99); 29 Oct 2018 14:09:43 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 29 Oct 2018 14:09:43 +0000 From: GitBox To: commits@airflow.apache.org Subject: [GitHub] zackmeso opened a new pull request #4114: AIRFLOW-3259] Fix internal server error when displaying charts Message-ID: <154082218278.13660.17282926098226708765.gitbox@gitbox.apache.org> Date: Mon, 29 Oct 2018 14:09:42 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit zackmeso opened a new pull request #4114: AIRFLOW-3259] Fix internal server error when displaying charts URL: https://github.com/apache/incubator-airflow/pull/4114 This is caused by the fact that the function 'sort' is no longer a part of Dataframe in pandas and is still used in the code base. It has ever since been replaced by 'sort_values'. Replacing the function gets the chart display back to its normal behaviour. Make sure you have checked _all_ steps below. ### Jira - [x] My PR addresses the following [Airflow Jira](https://issues.apache.org/jira/browse/AIRFLOW-3259/) issues and references them in the PR title. For example, "\[AIRFLOW-XXX\] My Airflow PR" - https://issues.apache.org/jira/browse/AIRFLOW-XXX - In case you are fixing a typo in the documentation you can prepend your commit with \[AIRFLOW-XXX\], code changes always need a Jira issue. ### Description - [x] Here are some details about my PR, including screenshots of any UI changes: **Problem** When you try to create a chart and display metrics you get an internal server error. **Cause** A dependency issue: the function "sort" was deprecated and is no longer a part of pandas Dataframe and thus it creates a problem when creating charts in airflow. **Solution** The sort function needs to replaced by sort_values instead (See code below) `# Replace this ` `df = df.sort(df.columns[0]) ` `# By this ` `df = df.sort_values(by=df.columns[0])` In views.py **Result** The data is loaded and the chart displayed ### Tests - [x] My PR does not need testing for this extremely good reason: My code doesn't change the behavior of the code, but rather modifies a no longer existing function (in pandas) with its new substitute. ### Commits - [x] My commit all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)": 1. Subject is separated from body by a blank line 1. Subject is limited to 50 characters (not including Jira issue reference) 1. Subject does not end with a period 1. Subject uses the imperative mood ("add", not "adding") 1. Body wraps at 72 characters 1. Body explains "what" and "why", not "how" ### Code Quality - [x] Passes `flake8` ---------------------------------------------------------------- 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 With regards, Apache Git Services