diff --git a/src/core_plugins/kibana/public/management/sections/indices/create_index_pattern_wizard/components/step_index_pattern/step_index_pattern.js b/src/core_plugins/kibana/public/management/sections/indices/create_index_pattern_wizard/components/step_index_pattern/step_index_pattern.js index d184c099e8fcc5..60b5cb2dd8daed 100644 --- a/src/core_plugins/kibana/public/management/sections/indices/create_index_pattern_wizard/components/step_index_pattern/step_index_pattern.js +++ b/src/core_plugins/kibana/public/management/sections/indices/create_index_pattern_wizard/components/step_index_pattern/step_index_pattern.js @@ -219,19 +219,12 @@ export class StepIndexPatternComponent extends Component { } iconType="help" color="warning" - > -

- -

-
+ /> ); } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/components/field_list/field_list.js b/x-pack/plugins/rollup/public/crud_app/sections/components/field_list/field_list.js index c528d11573c343..912b945a0d7e04 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/components/field_list/field_list.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/components/field_list/field_list.js @@ -54,6 +54,7 @@ export const FieldList = ({ toolsRight: addButton ? addButton : undefined, box: { incremental: true, + placeholder: 'Search', }, }; diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/components/index.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/components/index.js index 6aa485d7bfb822..e5c8eb2e2e17f1 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/components/index.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/components/index.js @@ -6,3 +6,4 @@ export { FieldChooser } from './field_chooser'; export { CronEditor } from './cron_editor'; +export { StepError } from './step_error'; diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/components/step_error.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/components/step_error.js new file mode 100644 index 00000000000000..487f2d24038980 --- /dev/null +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/components/step_error.js @@ -0,0 +1,36 @@ +/* + * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one + * or more contributor license agreements. Licensed under the Elastic License; + * you may not use this file except in compliance with the Elastic License. + */ + +import React, { Fragment } from 'react'; +import PropTypes from 'prop-types'; +import { FormattedMessage } from '@kbn/i18n/react'; + +import { + EuiCallOut, + EuiSpacer, +} from '@elastic/eui'; + +export function StepError({ title = ( + +) }) { + return ( + + + + + ); +} + +StepError.propTypes = { + title: PropTypes.node, +}; diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_date_histogram.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_date_histogram.js index 743b871b8f8b1f..2ef947c4444ce4 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_date_histogram.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_date_histogram.js @@ -11,7 +11,6 @@ import moment from 'moment-timezone'; import { EuiButtonEmpty, - EuiCallOut, EuiDescribedFormGroup, EuiFieldText, EuiFlexGroup, @@ -34,6 +33,8 @@ import { dateHistogramAggregationUrl, } from '../../../services'; +import { StepError } from './components'; + const timeZoneOptions = moment.tz.names().map(name => ({ value: name, text: name, @@ -85,7 +86,7 @@ export class StepDateHistogramUi extends Component {

@@ -98,7 +99,7 @@ export class StepDateHistogramUi extends Component {

@@ -114,7 +115,7 @@ export class StepDateHistogramUi extends Component {

@@ -127,7 +128,7 @@ export class StepDateHistogramUi extends Component {

@@ -140,7 +141,7 @@ export class StepDateHistogramUi extends Component {

@@ -158,7 +159,7 @@ export class StepDateHistogramUi extends Component {

@@ -247,10 +248,7 @@ export class StepDateHistogramUi extends Component {

@@ -282,7 +280,7 @@ export class StepDateHistogramUi extends Component { label={( )} error={errorDateHistogramInterval} @@ -331,21 +329,7 @@ export class StepDateHistogramUi extends Component { return null; } - return ( - - - - )} - color="danger" - iconType="cross" - /> - - ); + return ; } } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_histogram.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_histogram.js index 2d921c7dd30dad..edcd8109ec5d37 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_histogram.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_histogram.js @@ -10,7 +10,6 @@ import { injectI18n, FormattedMessage } from '@kbn/i18n/react'; import { EuiButtonEmpty, - EuiCallOut, EuiDescribedFormGroup, EuiFieldNumber, EuiFlexGroup, @@ -31,6 +30,7 @@ import { import { FieldChooser, + StepError, } from './components'; export class StepHistogramUi extends Component { @@ -98,7 +98,7 @@ export class StepHistogramUi extends Component {

@@ -230,21 +230,7 @@ export class StepHistogramUi extends Component { return null; } - return ( - - - - )} - color="danger" - iconType="cross" - /> - - ); + return ; } } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_logistics.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_logistics.js index bcb3222ca86178..8cc55499e0923b 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_logistics.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_logistics.js @@ -10,7 +10,6 @@ import { injectI18n, FormattedMessage } from '@kbn/i18n/react'; import { EuiButtonEmpty, - EuiCallOut, EuiDescribedFormGroup, EuiFieldNumber, EuiFieldText, @@ -28,7 +27,7 @@ import { import { INDEX_PATTERN_ILLEGAL_CHARACTERS_VISIBLE } from 'ui/index_patterns'; import { INDEX_ILLEGAL_CHARACTERS_VISIBLE } from 'ui/indices'; import { logisticalDetailsUrl, cronUrl } from '../../../services'; -import { CronEditor } from './components'; +import { CronEditor, StepError } from './components'; const indexPatternIllegalCharacters = INDEX_PATTERN_ILLEGAL_CHARACTERS_VISIBLE.join(' '); const indexIllegalCharacters = INDEX_ILLEGAL_CHARACTERS_VISIBLE.join(' '); @@ -99,7 +98,7 @@ export class StepLogisticsUi extends Component {

); @@ -111,14 +110,14 @@ export class StepLogisticsUi extends Component {

* }} />

{indexPatternIllegalCharacters} }} />

@@ -152,7 +151,7 @@ export class StepLogisticsUi extends Component { label={( )} error={errorRollupCron} @@ -167,7 +166,7 @@ export class StepLogisticsUi extends Component { ) }} @@ -219,7 +218,7 @@ export class StepLogisticsUi extends Component { @@ -272,7 +271,7 @@ export class StepLogisticsUi extends Component {

@@ -350,7 +349,7 @@ export class StepLogisticsUi extends Component { description={( )} fullWidth @@ -388,7 +387,7 @@ export class StepLogisticsUi extends Component { helpText={( {indexIllegalCharacters} }} /> )} @@ -418,7 +417,7 @@ export class StepLogisticsUi extends Component { )} @@ -433,7 +432,7 @@ export class StepLogisticsUi extends Component {
@@ -443,7 +442,7 @@ export class StepLogisticsUi extends Component { id="xpack.rollupJobs.create.stepLogistics.sectionPageSizeDescription" defaultMessage={` A larger page size - will roll up data more quickly, but will require more memory during processing. + will roll up data quicker, but requires more memory. `} /> )} @@ -485,8 +484,9 @@ export class StepLogisticsUi extends Component { )} @@ -496,7 +496,7 @@ export class StepLogisticsUi extends Component { label={( )} error={errorRollupDelay} @@ -506,7 +506,7 @@ export class StepLogisticsUi extends Component {

@@ -535,21 +535,7 @@ export class StepLogisticsUi extends Component { return null; } - return ( - - - - )} - color="danger" - iconType="cross" - /> - - ); + return ; } } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_metrics.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_metrics.js index 3a1788682101a1..5dae20b2e889ee 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_metrics.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_metrics.js @@ -10,7 +10,6 @@ import { injectI18n, FormattedMessage } from '@kbn/i18n/react'; import { EuiButtonEmpty, - EuiCallOut, EuiCheckbox, EuiFlexGroup, EuiFlexItem, @@ -29,6 +28,7 @@ import { import { FieldChooser, + StepError, } from './components'; export class StepMetricsUi extends Component { @@ -190,8 +190,8 @@ export class StepMetricsUi extends Component {

@@ -252,16 +252,7 @@ export class StepMetricsUi extends Component { return null; } - return ( - - - - - ); + return ; } } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_terms.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_terms.js index 8df9da5724d37d..07673aea04056a 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_terms.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps/step_terms.js @@ -98,8 +98,8 @@ export class StepTermsUi extends Component { id="xpack.rollupJobs.create.stepTermsDescription" defaultMessage={` Select the fields you want to bucket using terms aggregations. This can be - potentially costly for high-cardinality groups such as IP addresses, especially - if the time-bucket is particularly sparse. + costly for high-cardinality fields such as IP addresses, + if the time bucket is sparse. `} />

diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/index.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/index.js index 29629c1eb35076..20bdb6402d40db 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/index.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/index.js @@ -44,7 +44,9 @@ export const stepIdToStepConfigMap = { // Every week on Saturday, at 00:00:00 rollupCron: '0 0 0 ? * 7', simpleRollupCron: '0 0 0 ? * 7', - rollupPageSize: '', + // The best page size boils down to how much memory the user has, e.g. how many buckets should + // be accumulated at one time. 1000 is probably a safe size without being too small. + rollupPageSize: 1000, // Though the API doesn't require a delay, in many real-world cases, servers will go down for // a few hours as they're being restarted. A delay of 1d would allow them that period to reboot // and the "expense" is pretty negligible in most cases: 1 day of extra non-rolled-up data. diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_field.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_field.js index 2e1c2628f10547..16400a5dcfc3a7 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_field.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_field.js @@ -12,7 +12,7 @@ export function validateDateHistogramField(dateHistogramField) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_interval.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_interval.js index 1fc275e4f607fb..c1180078174987 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_interval.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_date_histogram_interval.js @@ -17,7 +17,7 @@ export function validateDateHistogramInterval(dateHistogramInterval) { return [( )]; } @@ -29,7 +29,7 @@ export function validateDateHistogramInterval(dateHistogramInterval) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_histogram_interval.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_histogram_interval.js index 57d52984ae3419..468e4b067cfceb 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_histogram_interval.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_histogram_interval.js @@ -17,7 +17,7 @@ export function validateHistogramInterval(histogram, histogramInterval) { return [( )]; } @@ -26,7 +26,7 @@ export function validateHistogramInterval(histogram, histogramInterval) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_id.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_id.js index ca67ab9e22a404..9cfa1fb5eebe33 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_id.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_id.js @@ -12,7 +12,7 @@ export function validateId(id) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_index_pattern.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_index_pattern.js index 97a9bccc706231..fcf8ea0ad51304 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_index_pattern.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_index_pattern.js @@ -14,7 +14,7 @@ export function validateIndexPattern(indexPattern, rollupIndex) { return [( )]; } @@ -23,7 +23,7 @@ export function validateIndexPattern(indexPattern, rollupIndex) { return [( )]; } @@ -40,7 +40,7 @@ export function validateIndexPattern(indexPattern, rollupIndex) { return [( {illegalCharacters.join(' ')} }} /> )]; @@ -50,7 +50,7 @@ export function validateIndexPattern(indexPattern, rollupIndex) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_metrics.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_metrics.js index 815585941c056f..cf07d57dc1860a 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_metrics.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_metrics.js @@ -22,7 +22,8 @@ export function validateMetrics(metrics) { return [( )]; diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_cron.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_cron.js index a508de7cda03f0..4df3ab8ef774d2 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_cron.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_cron.js @@ -13,7 +13,7 @@ export function validateRollupCron(rollupCron) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_delay.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_delay.js index 52690f4b566aae..26f6d0bdfcfc71 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_delay.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_delay.js @@ -25,7 +25,7 @@ export function validateRollupDelay(rollupDelay) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_index.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_index.js index cb7d99400b3fdb..a5e6271e871823 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_index.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_index.js @@ -14,7 +14,7 @@ export function validateRollupIndex(rollupIndex, indexPattern) { return [( )]; } @@ -23,7 +23,7 @@ export function validateRollupIndex(rollupIndex, indexPattern) { return [( )]; } @@ -40,7 +40,7 @@ export function validateRollupIndex(rollupIndex, indexPattern) { return [( {illegalCharacters.join(' ')} }} /> )]; @@ -50,7 +50,7 @@ export function validateRollupIndex(rollupIndex, indexPattern) { return [( )]; } @@ -59,7 +59,7 @@ export function validateRollupIndex(rollupIndex, indexPattern) { return [( )]; } @@ -68,7 +68,7 @@ export function validateRollupIndex(rollupIndex, indexPattern) { return [( )]; } diff --git a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_page_size.js b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_page_size.js index c371578e4e5dec..8ebbdb2d3f5dc6 100644 --- a/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_page_size.js +++ b/x-pack/plugins/rollup/public/crud_app/sections/job_create/steps_config/validate_rollup_page_size.js @@ -12,7 +12,7 @@ export function validateRollupPageSize(rollupPageSize) { return [( )]; } @@ -21,7 +21,7 @@ export function validateRollupPageSize(rollupPageSize) { return [( )]; }