----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62119/#review184790 -----------------------------------------------------------
contrib/views/files/src/main/resources/ui/app/controllers/files.js Line 81 (original), 74 (patched) <https://reviews.apache.org/r/62119/#comment260989> I guess you donot need to have a computed property on validSearchText and sortedContent. contrib/views/files/src/main/resources/ui/app/routes/files.js Line 46 (original), 46 (patched) <https://reviews.apache.org/r/62119/#comment260990> Please use a query parameter and reload the route. - DIPAYAN BHOWMICK On Sept. 7, 2017, 6:36 a.m., venkat sairam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62119/ > ----------------------------------------------------------- > > (Updated Sept. 7, 2017, 6:36 a.m.) > > > Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Nitiraj Rathore, > Padma Priya N, Pallav Kulshreshtha, and Prabhjyot Singh. > > > Bugs: AMBARI-21890 > https://issues.apache.org/jira/browse/AMBARI-21890 > > > Repository: ambari > > > Description > ------- > > A configurable limit have been imposed on the no of files being retrieved > from the server and the limit by default is set to 5000. > Search has been provided to find the file faster. > > > Diffs > ----- > > > contrib/views/commons/src/main/java/org/apache/ambari/view/commons/hdfs/FileOperationService.java > d6e484d > contrib/views/files/src/main/resources/ui/app/components/file-search.js > b65749c > contrib/views/files/src/main/resources/ui/app/controllers/files.js 8b5bb7b > contrib/views/files/src/main/resources/ui/app/routes/files.js 140732f > > contrib/views/files/src/main/resources/ui/app/templates/components/file-search.hbs > 298d672 > contrib/views/files/src/main/resources/ui/app/templates/files.hbs 63e0dd8 > > contrib/views/files/src/test/java/org/apache/ambari/view/filebrowser/FilebrowserTest.java > f431f66 > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/DirListInfo.java > PRE-CREATION > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/DirStatus.java > PRE-CREATION > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/HdfsApi.java > 90fa483 > > contrib/views/utils/src/test/java/org/apache/ambari/view/utils/hdfs/HdfsApiTest.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/62119/diff/2/ > > > Testing > ------- > > Manual testing done. > > > Thanks, > > venkat sairam > >