From 169274729e831e58ccf3d0ae01e34e853735d85a Mon Sep 17 00:00:00 2001
From: alanna scott <alanna.scott@airbnb.com>
Date: Thu, 20 Apr 2017 12:13:34 -0700
Subject: [PATCH] don't default sort by to first column

---
 .../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 c336148ba52b0..23b6906e5788f 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);