zero state for instances without collections

progress on istance properties

static styling for instance properties in sidebar

[wip] port, enterprise/community, reload missing

zero state for instances without collections

styling, changed default window size

moved sampling-message into refine view

Author: marcgurney

src/collection-stats/index.js

@@ -34,7 +34,7 @@ var CollectionStatsView = AmpersandView.extend({
   format: function(propertyName) {
     var value = this.model.get(propertyName) || 0;
     var precision = value <= 1000 ? '0' : '0.0';
-    var format = propertyName.indexOf('_size') > -1 ? 'b' : 'a';
+    var format = propertyName.indexOf('_size') > -1 ? ' b' : 'a';
     return numeral(value).format(precision + format);
   },
   derived: {

src/collection-stats/index.less

@@ -4,42 +4,59 @@
   position: absolute;
   right: 20px;
   text-align: right;
-  min-width: 450px;
+  min-width: 520px;
 }
 .collection-stats {
   display: inline-block;
+  position: relative;
   font-weight: 200;
   list-style: none;
-  padding: 0 30px 0 0;
-  margin: 11px 30px 0 0;
-  border-right: 1px solid @gray7;
+  padding: 0 20px 0 0;
+  margin: 0 20px 0 0;
+
+  &::after {
+    content: '';
+    display: block;
+    position: absolute;
+    right: 0;
+    top: 2px;
+    width: 1px;
+    height: 24px;
+    background: @gray7;
+  }
 
   &:last-child {
-    border-right: none;
     padding: 0;
     margin: 11px 0 0 0;
+
+    &::after {
+      display: none;
+    }
   }
 }
 .collection-stats-item {
   display: inline-block;
-  margin-right: 10px;
+  margin-right: 12px;
 
   &:last-child {
     margin-right: 0;
   }
 }
 .collection-stats-primary-label {
+  display: inline-block;
   text-transform: uppercase;
-  letter-spacing: 1px;
   font-size: 12px;
   color: @gray3;
+  margin-right: 5px;
 }
 .collection-stats-primary-value {
-  font-size: 36px;
-  line-height: 36px;
+  display: inline-block;
+  font-size: 24px;
+  line-height: 24px;
 }
 .collection-stats-label {
   font-size: 11px;
+  line-height: 11px;
   color: @gray3;
 }
 .collection-stats-value {

src/field-list/field.jade

@@ -1,5 +1,4 @@
 .schema-field.schema-field-basic
-  hr.field-divider
   .row
     .col-sm-4
       .schema-field-name
@@ -10,3 +9,4 @@
       div(data-hook='minichart-container')
   div(data-hook='fields-subview')
   div(data-hook='arrayfields-subview')
+  hr.field-divider

src/home/collection.jade

@@ -1,5 +1,4 @@
 .collection-view.clearfix
-  div(data-hook='sampling-message-subview')
   div.modal.fade(tabindex='-1', role='dialog', arialabelledby='Share Schema Confirmation', data-hook='share-schema-confirmation')
     div.modal-dialog.modal-sm
       .modal-content

src/home/collection.js

@@ -3,7 +3,6 @@ var CollectionStatsView = require('../collection-stats');
 var FieldListView = require('../field-list');
 var DocumentListView = require('../document-list');
 var RefineBarView = require('../refine-view');
-var SamplingMessageView = require('../sampling-message');
 var MongoDBCollection = require('../models/mongodb-collection');
 var SampledSchema = require('../models/sampled-schema');
 var app = require('ampersand-app');
@@ -161,15 +160,6 @@ var MongoDBCollectionView = View.extend({
           collection: this.schema.documents
         });
       }
-    },
-    sampling_message: {
-      hook: 'sampling-message-subview',
-      prepareView: function(el) {
-        return new SamplingMessageView({
-          el: el,
-          parent: this
-        });
-      }
     }
   }
 });

src/home/index.jade

@@ -4,4 +4,9 @@
     div.report-zero-state(data-hook='report-zero-state')
       img(src='images/zero-state-arrow-collections.png', width="110")
       span Choose a collection to analyze
+    div.no-collections-zero-state(data-hook='no-collections-zero-state')
+      span
+        |The MongoDB instance you are connected to does not contain any collections. 
+        a.show-connect-window Connect to another instance
+        |.
   div(data-hook='sidebar')

src/home/index.js

@@ -16,18 +16,30 @@ var HomeView = View.extend({
       allowNull: true,
       default: null
     },
-    showZeroState: {
+    showDefaultZeroState: {
       type: 'boolean',
-      default: true
+      default: false
+    },
+    showNoCollectionsZeroState: {
+      type: 'boolean',
+      default: false
     }
   },
   bindings: {
-    showZeroState: {
+    showDefaultZeroState: {
       hook: 'report-zero-state',
       type: 'booleanClass',
       no: 'hidden'
+    },
+    showNoCollectionsZeroState: {
+      hook: 'no-collections-zero-state',
+      type: 'booleanClass',
+      no: 'hidden'
     }
   },
+  events: {
+    'click a.show-connect-window': 'onClickShowConnectWindow'
+  },
   initialize: function() {
     this.listenTo(app.instance, 'sync', this.onInstanceFetched);
     this.listenTo(app.connection, 'change:name', this.updateTitle);
@@ -36,6 +48,11 @@ var HomeView = View.extend({
     app.instance.fetch();
   },
   onInstanceFetched: function() {
+    if(_.size(app.instance.collections) === 0) {
+      this.showNoCollectionsZeroState = true;
+    } else {
+      this.showDefaultZeroState = true;
+    }
     if (!this.ns) {
       app.instance.collections.unselectAll();
     } else {
@@ -58,11 +75,14 @@ var HomeView = View.extend({
 
     this.ns = model.getId();
     this.updateTitle(model);
-    this.showZeroState = false;
+    this.showDefaultZeroState = false;
     app.navigate(format('schema/%s', model.getId()), {
       silent: true
     });
   },
+  onClickShowConnectWindow: function() {
+    // code to close current connection window and open connect dialog
+  },
   template: require('./index.jade'),
   subviews: {
     _collection: {

src/home/index.less

@@ -53,7 +53,7 @@
 }
 
 .report-zero-state {
-  margin: 124px 0 0 20px;
+  margin: 190px 0 0 20px;
 
   img {
     margin-bottom: 7px;
@@ -65,13 +65,28 @@
     margin-left: 15px;
   }
 }
+.no-collections-zero-state {
+  text-align: center;
+  margin-top: 200px;
+  padding: 50px 75px;
+  font-size: 18px;
+  color: @gray4;
+  font-weight: 200;
+
+  a {
+    cursor: pointer;
+  }
+}
 
 .collection-view {
   header {
-    padding: 12px 25px;
+    padding: 12px 25px 0;
     position: relative;
     z-index: 1;
   }
+  h1 {
+    margin-top: 0;
+  }
   .column-container {
     display: flex;
     overflow: hidden;

src/models/sampled-schema.js

@@ -6,8 +6,24 @@ var filterableMixin = require('ampersand-collection-filterable');
 var SampledDocumentCollection = require('./sampled-document-collection');
 var es = require('event-stream');
 var debug = require('debug')('scout:models:schema');
+var debugMetrics = require('debug')('scout:metrics');
 var app = require('ampersand-app');
 
+// @todo: stub for metrics module. currently just logs debug messages with scout:metrics marker
+var metrics = {
+  track: function(label, err, obj) {
+    if (!obj) {
+      obj = err;
+      err = null;
+    }
+    if (err) {
+      debugMetrics('metrics error: ', err, obj);
+    } else {
+      debugMetrics('metrics log: ', obj);
+    }
+  }
+};
+
 /**
  * wrapping mongodb-schema's FieldCollection with a filterable mixin
  */
@@ -25,6 +41,8 @@ module.exports = Schema.extend({
     }
   },
   session: {
+    // total number of documents counted under the given query
+    total: 'number',
     is_fetching: {
       type: 'boolean',
       default: false
@@ -96,43 +114,93 @@ module.exports = Schema.extend({
     var model = this;
     wrapError(this, options);
 
-    var parse = function(doc, cb) {
-      model.parse(doc);
-      cb(null, doc);
+    var success = options.success;
+    options.success = function(resp) {
+      if (success) {
+        success(model, resp, options);
+      }
+      model.trigger('sync');
     };
+    var start = new Date();
+    var timeAtFirstDoc;
+    var erroredOnDocs = [];
 
-    var docs = [];
+    // No results found
+    var onEmpty = function() {
+      model.is_fetching = false;
+      model.documents.reset();
+      model.documents.trigger('sync');
+      options.success({});
+    };
 
+    var docs = [];
     var addToDocuments = function(doc, cb) {
       docs.push(doc);
       cb();
     };
 
+    var parse = function(doc, cb) {
+      if (!timeAtFirstDoc) {
+        timeAtFirstDoc = new Date();
+      }
+      try {
+        model.parse(doc);
+      } catch (err) {
+        erroredOnDocs.push(doc);
+        metrics.track('Schema: Error: Parse', err, {
+          doc: doc,
+          schema: model.serialize()
+        });
+      }
+      cb(null, doc);
+    };
+
     var onEnd = function(err) {
       model.is_fetching = false;
       if (err) {
+        metrics.track('Schema: Error: End', err, {
+          schema: model
+        });
         return options.error(model, err);
       }
       model.documents.reset(docs);
       model.documents.trigger('sync');
-      options.success({});
-    };
 
-    var success = options.success;
-    options.success = function(resp) {
-      if (success) {
-        success(model, resp, options);
-      }
-      model.trigger('sync');
+      // @note (imlucas): Any other metrics?  Feedback on `Schema *`?
+      metrics.track('Schema: Complete', {
+        Duration: new Date() - start,
+        'Total Document Count': model.total,
+        'Document Count': model.documents,
+        'Errored Document Count': erroredOnDocs.length,
+        'Time to First Doc': timeAtFirstDoc - start,
+        'Schema Height': model.height, // # of top level keys
+        'Schema Width': model.width, // max nesting depth
+        'Schema Sparsity': model.sparsity // lots of fields missing or consistent
+      });
+      options.success({});
     };
 
     model.trigger('request', {}, {}, options);
 
-    debug('creating sample stream');
-    app.client.sample(model.ns, options)
-      .pipe(es.map(parse))
-      .pipe(es.map(addToDocuments))
-      .pipe(es.wait(onEnd));
+    app.client.count(model.ns, options, function(err, count) {
+      if (err) {
+        metrics.track('Schema: Error: Count', err, {
+          schema: model
+        });
+        return options.error(model, err);
+      }
+      debug('options', options, 'count', count.count);
+      model.total = count.count;
+      if (model.total === 0) {
+        return onEmpty();
+      }
+
+      debug('creating sample stream');
+      app.client.sample(model.ns, options)
+        .pipe(es.map(parse))
+        .pipe(es.map(addToDocuments))
+        .pipe(es.wait(onEnd));
+    });
   },
   serialize: function() {
     var res = this.getAttributes({

src/refine-view/index.jade

@@ -1,6 +1,10 @@
-.refine-view-container: .row: .col-md-12
-  form: .input-group(data-hook='refine-input-group')
+.refine-view-container
+  .query-input-container: .row: .col-md-12: form: .input-group(data-hook='refine-input-group')
     input.form-control.input-sm(type='text', data-hook='refine-input')
     span.input-group-btn
       button.btn.btn-info.btn-sm(type='button', data-hook='refine-button') Apply
       button.btn.btn-default.btn-sm(type='button', data-hook='reset-button') Reset
+  div(data-hook='sampling-message-subview')
+
+  //- .sampling-message
+  //-   span Query returned 9867 documents. This report is based on a sample of 1000 documents (10.13%).