From 84fa0d19407d7fe662ea1945bb0f9e2570945e50 Mon Sep 17 00:00:00 2001
From: Alanna Scott <alanna.scott@airbnb.com>
Date: Thu, 20 Apr 2017 12:34:35 -0700
Subject: [PATCH] don't default sort by to first column (#2653)

---
 .../components/FilterableTable/FilterableTable.jsx          | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/superset/assets/javascripts/components/FilterableTable/FilterableTable.jsx b/superset/assets/javascripts/components/FilterableTable/FilterableTable.jsx
index c336148ba5..23b6906e57 100644
--- a/superset/assets/javascripts/components/FilterableTable/FilterableTable.jsx
+++ b/superset/assets/javascripts/components/FilterableTable/FilterableTable.jsx
@@ -41,7 +41,7 @@ export default class FilterableTable extends PureComponent {
       .reduce((curr, next) => curr + next);
 
     this.state = {
-      sortBy: props.orderedColumnKeys[0],
+      sortBy: null,
       sortDirection: SortDirection.ASC,
       fitted: false,
     };
@@ -142,9 +142,11 @@ export default class FilterableTable extends PureComponent {
       sortedAndFilteredList = this.list.filter(row => this.hasMatch(filterText, row));
     }
     // sort list
-    sortedAndFilteredList = sortedAndFilteredList
+    if (sortBy) {
+      sortedAndFilteredList = sortedAndFilteredList
       .sortBy(item => item[sortBy])
       .update(list => sortDirection === SortDirection.DESC ? list.reverse() : list);
+    }
 
     const rowGetter = ({ index }) => this.getDatum(sortedAndFilteredList, index);