NPE when running "help" when argument is marked as hidden #51
Comments
SerialVelocity
added a commit
to palantir/atlasdb
that referenced
this issue
Jul 22, 2016
SerialVelocity
added a commit
to palantir/atlasdb
that referenced
this issue
Aug 5, 2016
SerialVelocity
added a commit
to palantir/atlasdb
that referenced
this issue
Aug 16, 2016
SerialVelocity
added a commit
to palantir/atlasdb
that referenced
this issue
Aug 17, 2016
rjullman
added a commit
to palantir/atlasdb
that referenced
this issue
Aug 23, 2016
* Nit: use single-element ImmutableSet for clarity * SweepTaskRunnerImpl.getTimestampsToSweep: minor refactor * 0.11.0 release notes edits * Add ETE test for Atlas configured with no leader * Remove wrong config value from ETE tests * Refactor out on-demand loading and add tests for it. Clean-up files, convert/refactor tests to assertJ * Switch error to warn since it isn't actually an error * Rename SweepStrategySweeper to StrategySweeper * Get rid of NOTHING shortcut (essentially the same atm) * Make timestamp test work with all KVSes * Add release notes * Make test more consistent for DB KVS * StrategySweeper -> Sweeper, new test to maker sure we don't call the kvs when there aren't any rows to sweep * Get rid of Sweep at the beginning of two classes * Log messages should use AtlasDB * move to next release * Fix weird compile error * Remove bad optional.absent check * Update circle.yml to include container-logs in artifacts * Add split out logs for ETE tests * 0.11.1 release note * release notes for 0.11.2 * Add CassandraServerVersion with tests * 0.11.3 release notes * Update release-notes.rst * Test for cassandra duplicate table creation bug * Change the copyright header to use the regular one * Passwords should not be output in generated toString() methods Prevents an issue seen where a stack trace helpfully tries to print the config but also prints the password to the log files. Updated copyright of all modified classes to 2016. * Bump version * dont log loading a tracing prefs that isn't enabled at some point we started shipping an example prefs file that always gets loaded, so you get this 'error' message at startup by default now. * cleanup temp tables Necessary for a corresponding internal change. The general gist is open atlas never wanted this application-specific garbage anyways, so I've mostly removed it here and made it so someone can just implement an interface. Real reason for all of this is that we're moving to lazily initialized temp tables because postgres' rubbish temp table creation speeds screw with Hikari pool resizing, among other things. * Switch to DockerComposeRule instead of DockerComposition * Switch to DockerComposeRule instead of DockerComposition for TimelockServer * Switch to DockerComposeRule instead of DockerComposition for EteSetup * Switch to DockerComposeRule instead of DockerComposition for DbkvsTestSuite * Adding diagnostics, as per Oracle Support * Rename composeRule -> docker, per naming convention in DCR example code * Add entry in ValueTypes table for UUID A UUID is just two longs (a 16-byte value). We encode them as big-endian, so range scans are supported. * Clean up explanation for VAR_LONG and VAR_SIGNED_LONG * Reword introductory paragraph * Update release notes for PR #755. * Add advice for contributing code * Add explanation about FIXED_LONG_LITTLE_ENDIAN * Use footnotes because they're prettier * Fix typos in contribution guidelines * don't always log sql errors especially ones that are expected errors / caller logic is set up to deal with the failure. Fixes #709 * Add comment explaining the version * Clean up tests * Switch to shared static logger * Adjust table widths so the footnotes don't spill over * Add additional logging * Unused imports * ssl: false should override the presence of an SslConfiguration We want to add a default SslConfiguration to our internal application. However, users may have explicitly disabled Ssl for communication with their Cassandra, which will be broken by the presence of the new block overriding the `ssl()` param.. Despite being deprecated, the `ssl()` param should override the presence of an ssl config, meaning that we can apply this without being a breaking change. * License * Fix copy-paste error * Upgrade checkstyle to 6.18 * Add .baseline files * Remove a few of the checkstyle/findbugs. Change import order to match existing order. Change copyright notice to BSD-3. Auto-add copyright notice in IntelliJ * Add baseline to the gradle files * Enable checkstyle and findbugs report extraction in CircleCI * CassandraTestSuite: Add a Classrule to ensure Cassandra images are created * Switch from whitelisted projects to blacklisted projects * Remove unused variable * Clean up docker-compose * CassandraSchemaLockTest: Automated test for detecting multiple table creation error * Cleanup * Add matchers to get better error messages * Improve error messages * Move the cassandra multinode test to a separate directory * Revert copy-paste error * Fix typo: SIGNED_VAR_LONG -> VAR_SIGNED_LONG * Move checkstyle version into versions file, remove META-INF exclude, and add comment about why we have the findbugs excludes * Ensure C* image is created before docker-compose * Remove stray empty lines * Prep and enable baseline for atlasdb-dropwizard-bundle * Enable baseline for atlasdb-hikari * Prep atlasdb-api for baseline * Prep atlasdb-api tests for baseline * Fix-up findbugs errors for atlasdb-api * Enable baseline for atlasdb-api * Run cassandra-multinode-tests in a separate container * Create fromBoolean method in LockLeader enum * Docs: cassandra_KVS_config should have more information on ssl and leader block. * Improve the wording in docs * Add release notes * Upgrade to gradle 2.14.1 and jacoco to 0.7.7.201606060606. Fixes #767 * Revert "Test for cassandra duplicate table creation bug" * Revert "Revert "Test for cassandra duplicate table creation bug"" * Add container logs directory * Add .idea to .gitignore * Move to package, blacklist * Increase timeout * Implement the timestamp dropwizard command * Convert timestamp CLI dropwizard class into a generic one for all CLIs * Fix checkstyle and similar * Add temporary workaround for airlift/airline#51 * Convert leader blocks into TS/LOCK blocks * Refactor configuration conversion and argument extraction out * Add offline flag to CLIs * Add offline support to console and propagate it through for CLIs * Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure * Update nullables and unused import * Add missing copyright * Revert "Update nullables and unused import" This reverts commit 8ee3b4d. * Revert "Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure" This reverts commit 5248e3f. * Parameterise TestTimestampCommand * Further refactoring of timestamp command tests * Implement some ETE dropwizard tests * Refactor runCommand to take in a String * Remove random generated files * Add tests for AtlasDbCommandUtils * Ignore dropwizard ETE tests until docker-compose-rule#95 is merged and a new version is tagged * Added dropwizard bundle to release notes * Clean-up some code, fix an EMPTY_NAMESPACE bug when serialising/deserialising * Switch to docker run and re-enable tests * Add @nullable to AtlasDbConfigs * Set of minor modifications * Set of minor modifications * Annotate why we JsonIgnore onAcquireConnectionVisitor * Add tests for AtlasDbCliCommand#getOptionTypesForCommandMetadata * Refactor out addOptionToParser and add tests * Refactor out serialisation command * Fix-up checkstyle * Move timestamp file clean up into method * Add a new Leader configuration page * Make timeout 2 minutes * Fix checkstyle * Use correct jacoco version * Use assserThat from unit * Refactor CassandraSchemaLockTest and cassandra yml * Cleanup container logs * Cleanup build.gradle * Reduce timeout, extract CONTAINER_LOGS_DIRECTORY * minor doc nits * Prep atlasdb-dbkvs for baseline * Prepare atlasdb-dbkvs for findbugs * Enable baseline on atlasdb-dbkvs * Remove @after method * Increase timeout * Remove dependencies on ete1 as this is causing hanging builds. Fixed in #793 * Newline for Ben * Select the first leader according to alphabetical order to make sure lockCreator is deterministic. * Add tests * Fix CassandraConnectionTest by setting a valid quorumSize * Add docs change * Fix directory structure in atlasdb-impl-shared/src/test/java * Add profiling KVS wrapper to default kvs startup, use same order as internal product * Add necessary documentation for profiling kvs' * Docs nit: add comma before "which" * Add check to make sure it's possible to reach quorum * Wait for DB start * Prep atlasdb-impl-shared for baseline. Disable abbreviation checking in files with KV in their name. * Prep atlasdb-impl-shared tests for baseline * Get atlasdb-impl-shared and tests working with findbugs. Exclude DynamicPartitionMapImpl from synchronisation checks until later * Enable baseline for atlasdb-impl-shared * Code review fixes * Rename atlasdb-server -> atlasdb-service, atlasdb-server-dropwizard -> atlasdb-service-server * Add release notes * Prep atlasdb-cassandra for baseline * Fix-up some FindBugs for atlasdb-cassandra * Enable baseline on atlasdb-cassandra * Fix-up code review comments * ignore eclipse generated stuff * Reduce number of threads and timeout * Expand title line and fix link to file * Cleanup docs * Docs changes * Docs: Add link from Logging page to Enabling C* Tracing page. * Docs: minor fixes, mostly capitalisation * release notes edits for 0.12.0 * Rename link * Check for unique key constraints when inserting tables into metadata * Fix-up ETE dependencies so that we wait for the DB box to be created * another atlas -> atlasdb * Refactor DbkvsTestSuite so it runs more consistently locally * Remove test that checks duplicate table names throws. Cassandra doesn't throw, so we shouldn't really either * fix index.rst pages * edit breaking change note * final 0.12.0 release notes nits * Convert findbugs annotation version to a constant * Prep atlasdb-config for baseline * Enable baseline for atlasdb-config * Move release notes to right version * fix compile errors * publish test artifacts * add licenses * fix checkstyle errors * fixed errors from bad merge * fix checkstyle rule failures caused by merge * fix findbugs errors
rjullman
added a commit
to palantir/atlasdb
that referenced
this issue
Aug 23, 2016
* Update release-notes.rst * pgdev disallows printStackTrace * Fix broken link in docs * Use try-with-resources * Nit: use single-element ImmutableSet for clarity * SweepTaskRunnerImpl.getTimestampsToSweep: minor refactor * 0.11.0 release notes edits * Add ETE test for Atlas configured with no leader * Remove wrong config value from ETE tests * Refactor out on-demand loading and add tests for it. Clean-up files, convert/refactor tests to assertJ * Switch error to warn since it isn't actually an error * Rename SweepStrategySweeper to StrategySweeper * Get rid of NOTHING shortcut (essentially the same atm) * Make timestamp test work with all KVSes * Add release notes * Make test more consistent for DB KVS * StrategySweeper -> Sweeper, new test to maker sure we don't call the kvs when there aren't any rows to sweep * Get rid of Sweep at the beginning of two classes * Log messages should use AtlasDB * move to next release * Fix weird compile error * Remove bad optional.absent check * Update circle.yml to include container-logs in artifacts * Add split out logs for ETE tests * 0.11.1 release note * release notes for 0.11.2 * Add CassandraServerVersion with tests * 0.11.3 release notes * Update release-notes.rst * Test for cassandra duplicate table creation bug * Change the copyright header to use the regular one * Passwords should not be output in generated toString() methods Prevents an issue seen where a stack trace helpfully tries to print the config but also prints the password to the log files. Updated copyright of all modified classes to 2016. * Bump version * dont log loading a tracing prefs that isn't enabled at some point we started shipping an example prefs file that always gets loaded, so you get this 'error' message at startup by default now. * cleanup temp tables Necessary for a corresponding internal change. The general gist is open atlas never wanted this application-specific garbage anyways, so I've mostly removed it here and made it so someone can just implement an interface. Real reason for all of this is that we're moving to lazily initialized temp tables because postgres' rubbish temp table creation speeds screw with Hikari pool resizing, among other things. * Switch to DockerComposeRule instead of DockerComposition * Switch to DockerComposeRule instead of DockerComposition for TimelockServer * Switch to DockerComposeRule instead of DockerComposition for EteSetup * Switch to DockerComposeRule instead of DockerComposition for DbkvsTestSuite * Adding diagnostics, as per Oracle Support * Rename composeRule -> docker, per naming convention in DCR example code * Add entry in ValueTypes table for UUID A UUID is just two longs (a 16-byte value). We encode them as big-endian, so range scans are supported. * Clean up explanation for VAR_LONG and VAR_SIGNED_LONG * Reword introductory paragraph * Update release notes for PR #755. * Add advice for contributing code * Add explanation about FIXED_LONG_LITTLE_ENDIAN * Use footnotes because they're prettier * Fix typos in contribution guidelines * don't always log sql errors especially ones that are expected errors / caller logic is set up to deal with the failure. Fixes #709 * Add comment explaining the version * Clean up tests * Switch to shared static logger * Adjust table widths so the footnotes don't spill over * Add additional logging * Unused imports * ssl: false should override the presence of an SslConfiguration We want to add a default SslConfiguration to our internal application. However, users may have explicitly disabled Ssl for communication with their Cassandra, which will be broken by the presence of the new block overriding the `ssl()` param.. Despite being deprecated, the `ssl()` param should override the presence of an ssl config, meaning that we can apply this without being a breaking change. * License * Fix copy-paste error * Upgrade checkstyle to 6.18 * Add .baseline files * Remove a few of the checkstyle/findbugs. Change import order to match existing order. Change copyright notice to BSD-3. Auto-add copyright notice in IntelliJ * Add baseline to the gradle files * Enable checkstyle and findbugs report extraction in CircleCI * CassandraTestSuite: Add a Classrule to ensure Cassandra images are created * Switch from whitelisted projects to blacklisted projects * Remove unused variable * Clean up docker-compose * CassandraSchemaLockTest: Automated test for detecting multiple table creation error * Cleanup * Add matchers to get better error messages * Improve error messages * Move the cassandra multinode test to a separate directory * Revert copy-paste error * Fix typo: SIGNED_VAR_LONG -> VAR_SIGNED_LONG * Move checkstyle version into versions file, remove META-INF exclude, and add comment about why we have the findbugs excludes * Ensure C* image is created before docker-compose * Remove stray empty lines * Prep and enable baseline for atlasdb-dropwizard-bundle * Enable baseline for atlasdb-hikari * Prep atlasdb-api for baseline * Prep atlasdb-api tests for baseline * Fix-up findbugs errors for atlasdb-api * Enable baseline for atlasdb-api * Run cassandra-multinode-tests in a separate container * Create fromBoolean method in LockLeader enum * Docs: cassandra_KVS_config should have more information on ssl and leader block. * Improve the wording in docs * Add release notes * Upgrade to gradle 2.14.1 and jacoco to 0.7.7.201606060606. Fixes #767 * Revert "Test for cassandra duplicate table creation bug" * Revert "Revert "Test for cassandra duplicate table creation bug"" * Add container logs directory * Add .idea to .gitignore * Move to package, blacklist * Increase timeout * Implement the timestamp dropwizard command * Convert timestamp CLI dropwizard class into a generic one for all CLIs * Fix checkstyle and similar * Add temporary workaround for airlift/airline#51 * Convert leader blocks into TS/LOCK blocks * Refactor configuration conversion and argument extraction out * Add offline flag to CLIs * Add offline support to console and propagate it through for CLIs * Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure * Update nullables and unused import * Add missing copyright * Revert "Update nullables and unused import" This reverts commit 8ee3b4d. * Revert "Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure" This reverts commit 5248e3f. * Parameterise TestTimestampCommand * Further refactoring of timestamp command tests * Implement some ETE dropwizard tests * Refactor runCommand to take in a String * Remove random generated files * Add tests for AtlasDbCommandUtils * Ignore dropwizard ETE tests until docker-compose-rule#95 is merged and a new version is tagged * Added dropwizard bundle to release notes * Clean-up some code, fix an EMPTY_NAMESPACE bug when serialising/deserialising * Switch to docker run and re-enable tests * Add @nullable to AtlasDbConfigs * Set of minor modifications * Set of minor modifications * Annotate why we JsonIgnore onAcquireConnectionVisitor * Add tests for AtlasDbCliCommand#getOptionTypesForCommandMetadata * Refactor out addOptionToParser and add tests * Refactor out serialisation command * Fix-up checkstyle * Move timestamp file clean up into method * Add a new Leader configuration page * Make timeout 2 minutes * Fix checkstyle * Use correct jacoco version * Use assserThat from unit * Refactor CassandraSchemaLockTest and cassandra yml * Cleanup container logs * Cleanup build.gradle * Reduce timeout, extract CONTAINER_LOGS_DIRECTORY * minor doc nits * Prep atlasdb-dbkvs for baseline * Prepare atlasdb-dbkvs for findbugs * Enable baseline on atlasdb-dbkvs * Remove @after method * Increase timeout * Remove dependencies on ete1 as this is causing hanging builds. Fixed in #793 * Newline for Ben * Select the first leader according to alphabetical order to make sure lockCreator is deterministic. * Add tests * Fix CassandraConnectionTest by setting a valid quorumSize * Add docs change * Fix directory structure in atlasdb-impl-shared/src/test/java * Add profiling KVS wrapper to default kvs startup, use same order as internal product * Add necessary documentation for profiling kvs' * Docs nit: add comma before "which" * Add check to make sure it's possible to reach quorum * Wait for DB start * Prep atlasdb-impl-shared for baseline. Disable abbreviation checking in files with KV in their name. * Prep atlasdb-impl-shared tests for baseline * Get atlasdb-impl-shared and tests working with findbugs. Exclude DynamicPartitionMapImpl from synchronisation checks until later * Enable baseline for atlasdb-impl-shared * Code review fixes * Rename atlasdb-server -> atlasdb-service, atlasdb-server-dropwizard -> atlasdb-service-server * Add release notes * Prep atlasdb-cassandra for baseline * Fix-up some FindBugs for atlasdb-cassandra * Enable baseline on atlasdb-cassandra * Fix-up code review comments * ignore eclipse generated stuff * Reduce number of threads and timeout * Expand title line and fix link to file * Cleanup docs * Docs changes * Docs: Add link from Logging page to Enabling C* Tracing page. * Docs: minor fixes, mostly capitalisation * release notes edits for 0.12.0 * Rename link * Check for unique key constraints when inserting tables into metadata * Fix-up ETE dependencies so that we wait for the DB box to be created * another atlas -> atlasdb * Refactor DbkvsTestSuite so it runs more consistently locally * Remove test that checks duplicate table names throws. Cassandra doesn't throw, so we shouldn't really either * fix index.rst pages * edit breaking change note * final 0.12.0 release notes nits * Convert findbugs annotation version to a constant * Prep atlasdb-config for baseline * Enable baseline for atlasdb-config * Move release notes to right version * publish test artifacts * move atlas dagger instantiation into its own project (#816) * move atlas dagger instantiation into its own project * fix checkstyle failures * rename atlasdb-server to atlasdb-service * rename atlasdb-core to atlasdb-dagger * remove extraneous test.sh
rjullman
added a commit
to palantir/atlasdb
that referenced
this issue
Sep 8, 2016
* Nit: use single-element ImmutableSet for clarity * SweepTaskRunnerImpl.getTimestampsToSweep: minor refactor * 0.11.0 release notes edits * Add ETE test for Atlas configured with no leader * Remove wrong config value from ETE tests * Refactor out on-demand loading and add tests for it. Clean-up files, convert/refactor tests to assertJ * Switch error to warn since it isn't actually an error * Rename SweepStrategySweeper to StrategySweeper * Get rid of NOTHING shortcut (essentially the same atm) * Make timestamp test work with all KVSes * Add release notes * Make test more consistent for DB KVS * StrategySweeper -> Sweeper, new test to maker sure we don't call the kvs when there aren't any rows to sweep * Get rid of Sweep at the beginning of two classes * Log messages should use AtlasDB * move to next release * Fix weird compile error * Remove bad optional.absent check * Update circle.yml to include container-logs in artifacts * Add split out logs for ETE tests * 0.11.1 release note * release notes for 0.11.2 * Add CassandraServerVersion with tests * 0.11.3 release notes * Update release-notes.rst * Test for cassandra duplicate table creation bug * Change the copyright header to use the regular one * Passwords should not be output in generated toString() methods Prevents an issue seen where a stack trace helpfully tries to print the config but also prints the password to the log files. Updated copyright of all modified classes to 2016. * Bump version * dont log loading a tracing prefs that isn't enabled at some point we started shipping an example prefs file that always gets loaded, so you get this 'error' message at startup by default now. * cleanup temp tables Necessary for a corresponding internal change. The general gist is open atlas never wanted this application-specific garbage anyways, so I've mostly removed it here and made it so someone can just implement an interface. Real reason for all of this is that we're moving to lazily initialized temp tables because postgres' rubbish temp table creation speeds screw with Hikari pool resizing, among other things. * Switch to DockerComposeRule instead of DockerComposition * Switch to DockerComposeRule instead of DockerComposition for TimelockServer * Switch to DockerComposeRule instead of DockerComposition for EteSetup * Switch to DockerComposeRule instead of DockerComposition for DbkvsTestSuite * Adding diagnostics, as per Oracle Support * Rename composeRule -> docker, per naming convention in DCR example code * Add entry in ValueTypes table for UUID A UUID is just two longs (a 16-byte value). We encode them as big-endian, so range scans are supported. * Clean up explanation for VAR_LONG and VAR_SIGNED_LONG * Reword introductory paragraph * Update release notes for PR #755. * Add advice for contributing code * Add explanation about FIXED_LONG_LITTLE_ENDIAN * Use footnotes because they're prettier * Fix typos in contribution guidelines * don't always log sql errors especially ones that are expected errors / caller logic is set up to deal with the failure. Fixes #709 * Add comment explaining the version * Clean up tests * Switch to shared static logger * Adjust table widths so the footnotes don't spill over * Add additional logging * Unused imports * ssl: false should override the presence of an SslConfiguration We want to add a default SslConfiguration to our internal application. However, users may have explicitly disabled Ssl for communication with their Cassandra, which will be broken by the presence of the new block overriding the `ssl()` param.. Despite being deprecated, the `ssl()` param should override the presence of an ssl config, meaning that we can apply this without being a breaking change. * License * Fix copy-paste error * Upgrade checkstyle to 6.18 * Add .baseline files * Remove a few of the checkstyle/findbugs. Change import order to match existing order. Change copyright notice to BSD-3. Auto-add copyright notice in IntelliJ * Add baseline to the gradle files * Enable checkstyle and findbugs report extraction in CircleCI * CassandraTestSuite: Add a Classrule to ensure Cassandra images are created * Switch from whitelisted projects to blacklisted projects * Remove unused variable * Clean up docker-compose * CassandraSchemaLockTest: Automated test for detecting multiple table creation error * Cleanup * Add matchers to get better error messages * Improve error messages * Move the cassandra multinode test to a separate directory * Revert copy-paste error * Fix typo: SIGNED_VAR_LONG -> VAR_SIGNED_LONG * Move checkstyle version into versions file, remove META-INF exclude, and add comment about why we have the findbugs excludes * Ensure C* image is created before docker-compose * Remove stray empty lines * Prep and enable baseline for atlasdb-dropwizard-bundle * Enable baseline for atlasdb-hikari * Prep atlasdb-api for baseline * Prep atlasdb-api tests for baseline * Fix-up findbugs errors for atlasdb-api * Enable baseline for atlasdb-api * Run cassandra-multinode-tests in a separate container * Create fromBoolean method in LockLeader enum * Docs: cassandra_KVS_config should have more information on ssl and leader block. * Improve the wording in docs * Add release notes * Upgrade to gradle 2.14.1 and jacoco to 0.7.7.201606060606. Fixes #767 * Revert "Test for cassandra duplicate table creation bug" * Revert "Revert "Test for cassandra duplicate table creation bug"" * Add container logs directory * Add .idea to .gitignore * Move to package, blacklist * Increase timeout * Implement the timestamp dropwizard command * Convert timestamp CLI dropwizard class into a generic one for all CLIs * Fix checkstyle and similar * Add temporary workaround for airlift/airline#51 * Convert leader blocks into TS/LOCK blocks * Refactor configuration conversion and argument extraction out * Add offline flag to CLIs * Add offline support to console and propagate it through for CLIs * Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure * Update nullables and unused import * Add missing copyright * Revert "Update nullables and unused import" This reverts commit 8ee3b4d. * Revert "Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure" This reverts commit 5248e3f. * Parameterise TestTimestampCommand * Further refactoring of timestamp command tests * Implement some ETE dropwizard tests * Refactor runCommand to take in a String * Remove random generated files * Add tests for AtlasDbCommandUtils * Ignore dropwizard ETE tests until docker-compose-rule#95 is merged and a new version is tagged * Added dropwizard bundle to release notes * Clean-up some code, fix an EMPTY_NAMESPACE bug when serialising/deserialising * Switch to docker run and re-enable tests * Add @nullable to AtlasDbConfigs * Set of minor modifications * Set of minor modifications * Annotate why we JsonIgnore onAcquireConnectionVisitor * Add tests for AtlasDbCliCommand#getOptionTypesForCommandMetadata * Refactor out addOptionToParser and add tests * Refactor out serialisation command * Fix-up checkstyle * Move timestamp file clean up into method * Add a new Leader configuration page * Make timeout 2 minutes * Fix checkstyle * Use correct jacoco version * Use assserThat from unit * Refactor CassandraSchemaLockTest and cassandra yml * Cleanup container logs * Cleanup build.gradle * Reduce timeout, extract CONTAINER_LOGS_DIRECTORY * minor doc nits * Prep atlasdb-dbkvs for baseline * Prepare atlasdb-dbkvs for findbugs * Enable baseline on atlasdb-dbkvs * Remove @after method * Increase timeout * Remove dependencies on ete1 as this is causing hanging builds. Fixed in #793 * Newline for Ben * Select the first leader according to alphabetical order to make sure lockCreator is deterministic. * Add tests * Fix CassandraConnectionTest by setting a valid quorumSize * Add docs change * Fix directory structure in atlasdb-impl-shared/src/test/java * Add profiling KVS wrapper to default kvs startup, use same order as internal product * Add necessary documentation for profiling kvs' * Docs nit: add comma before "which" * Add check to make sure it's possible to reach quorum * Wait for DB start * Prep atlasdb-impl-shared for baseline. Disable abbreviation checking in files with KV in their name. * Prep atlasdb-impl-shared tests for baseline * Get atlasdb-impl-shared and tests working with findbugs. Exclude DynamicPartitionMapImpl from synchronisation checks until later * Enable baseline for atlasdb-impl-shared * Code review fixes * Rename atlasdb-server -> atlasdb-service, atlasdb-server-dropwizard -> atlasdb-service-server * Add release notes * Prep atlasdb-cassandra for baseline * Fix-up some FindBugs for atlasdb-cassandra * Enable baseline on atlasdb-cassandra * Fix-up code review comments * ignore eclipse generated stuff * Reduce number of threads and timeout * Expand title line and fix link to file * Cleanup docs * Docs changes * Docs: Add link from Logging page to Enabling C* Tracing page. * Docs: minor fixes, mostly capitalisation * release notes edits for 0.12.0 * Rename link * Check for unique key constraints when inserting tables into metadata * Fix-up ETE dependencies so that we wait for the DB box to be created * another atlas -> atlasdb * Refactor DbkvsTestSuite so it runs more consistently locally * Remove test that checks duplicate table names throws. Cassandra doesn't throw, so we shouldn't really either * fix index.rst pages * edit breaking change note * final 0.12.0 release notes nits * Convert findbugs annotation version to a constant * Prep atlasdb-config for baseline * Enable baseline for atlasdb-config * Move release notes to right version * fix compile errors * publish test artifacts * add licenses * fix checkstyle errors * fixed errors from bad merge * fix checkstyle rule failures caused by merge * fix findbugs errors
rjullman
added a commit
to palantir/atlasdb
that referenced
this issue
Sep 12, 2016
* Nit: use single-element ImmutableSet for clarity * SweepTaskRunnerImpl.getTimestampsToSweep: minor refactor * 0.11.0 release notes edits * Add ETE test for Atlas configured with no leader * Remove wrong config value from ETE tests * Refactor out on-demand loading and add tests for it. Clean-up files, convert/refactor tests to assertJ * Switch error to warn since it isn't actually an error * Rename SweepStrategySweeper to StrategySweeper * Get rid of NOTHING shortcut (essentially the same atm) * Make timestamp test work with all KVSes * Add release notes * Make test more consistent for DB KVS * StrategySweeper -> Sweeper, new test to maker sure we don't call the kvs when there aren't any rows to sweep * Get rid of Sweep at the beginning of two classes * Log messages should use AtlasDB * move to next release * Fix weird compile error * Remove bad optional.absent check * Update circle.yml to include container-logs in artifacts * Add split out logs for ETE tests * 0.11.1 release note * release notes for 0.11.2 * Add CassandraServerVersion with tests * 0.11.3 release notes * Update release-notes.rst * Test for cassandra duplicate table creation bug * Change the copyright header to use the regular one * Passwords should not be output in generated toString() methods Prevents an issue seen where a stack trace helpfully tries to print the config but also prints the password to the log files. Updated copyright of all modified classes to 2016. * Bump version * dont log loading a tracing prefs that isn't enabled at some point we started shipping an example prefs file that always gets loaded, so you get this 'error' message at startup by default now. * cleanup temp tables Necessary for a corresponding internal change. The general gist is open atlas never wanted this application-specific garbage anyways, so I've mostly removed it here and made it so someone can just implement an interface. Real reason for all of this is that we're moving to lazily initialized temp tables because postgres' rubbish temp table creation speeds screw with Hikari pool resizing, among other things. * Switch to DockerComposeRule instead of DockerComposition * Switch to DockerComposeRule instead of DockerComposition for TimelockServer * Switch to DockerComposeRule instead of DockerComposition for EteSetup * Switch to DockerComposeRule instead of DockerComposition for DbkvsTestSuite * Adding diagnostics, as per Oracle Support * Rename composeRule -> docker, per naming convention in DCR example code * Add entry in ValueTypes table for UUID A UUID is just two longs (a 16-byte value). We encode them as big-endian, so range scans are supported. * Clean up explanation for VAR_LONG and VAR_SIGNED_LONG * Reword introductory paragraph * Update release notes for PR #755. * Add advice for contributing code * Add explanation about FIXED_LONG_LITTLE_ENDIAN * Use footnotes because they're prettier * Fix typos in contribution guidelines * don't always log sql errors especially ones that are expected errors / caller logic is set up to deal with the failure. Fixes #709 * Add comment explaining the version * Clean up tests * Switch to shared static logger * Adjust table widths so the footnotes don't spill over * Add additional logging * Unused imports * ssl: false should override the presence of an SslConfiguration We want to add a default SslConfiguration to our internal application. However, users may have explicitly disabled Ssl for communication with their Cassandra, which will be broken by the presence of the new block overriding the `ssl()` param.. Despite being deprecated, the `ssl()` param should override the presence of an ssl config, meaning that we can apply this without being a breaking change. * License * Fix copy-paste error * Upgrade checkstyle to 6.18 * Add .baseline files * Remove a few of the checkstyle/findbugs. Change import order to match existing order. Change copyright notice to BSD-3. Auto-add copyright notice in IntelliJ * Add baseline to the gradle files * Enable checkstyle and findbugs report extraction in CircleCI * CassandraTestSuite: Add a Classrule to ensure Cassandra images are created * Switch from whitelisted projects to blacklisted projects * Remove unused variable * Clean up docker-compose * CassandraSchemaLockTest: Automated test for detecting multiple table creation error * Cleanup * Add matchers to get better error messages * Improve error messages * Move the cassandra multinode test to a separate directory * Revert copy-paste error * Fix typo: SIGNED_VAR_LONG -> VAR_SIGNED_LONG * Move checkstyle version into versions file, remove META-INF exclude, and add comment about why we have the findbugs excludes * Ensure C* image is created before docker-compose * Remove stray empty lines * Prep and enable baseline for atlasdb-dropwizard-bundle * Enable baseline for atlasdb-hikari * Prep atlasdb-api for baseline * Prep atlasdb-api tests for baseline * Fix-up findbugs errors for atlasdb-api * Enable baseline for atlasdb-api * Run cassandra-multinode-tests in a separate container * Create fromBoolean method in LockLeader enum * Docs: cassandra_KVS_config should have more information on ssl and leader block. * Improve the wording in docs * Add release notes * Upgrade to gradle 2.14.1 and jacoco to 0.7.7.201606060606. Fixes #767 * Revert "Test for cassandra duplicate table creation bug" * Revert "Revert "Test for cassandra duplicate table creation bug"" * Add container logs directory * Add .idea to .gitignore * Move to package, blacklist * Increase timeout * Implement the timestamp dropwizard command * Convert timestamp CLI dropwizard class into a generic one for all CLIs * Fix checkstyle and similar * Add temporary workaround for airlift/airline#51 * Convert leader blocks into TS/LOCK blocks * Refactor configuration conversion and argument extraction out * Add offline flag to CLIs * Add offline support to console and propagate it through for CLIs * Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure * Update nullables and unused import * Add missing copyright * Revert "Update nullables and unused import" This reverts commit 8ee3b4d. * Revert "Refector AtlasDbConfigs as part of #goldentime to play around with tell rather than ask code structure" This reverts commit 5248e3f. * Parameterise TestTimestampCommand * Further refactoring of timestamp command tests * Implement some ETE dropwizard tests * Refactor runCommand to take in a String * Remove random generated files * Add tests for AtlasDbCommandUtils * Ignore dropwizard ETE tests until docker-compose-rule#95 is merged and a new version is tagged * Added dropwizard bundle to release notes * Clean-up some code, fix an EMPTY_NAMESPACE bug when serialising/deserialising * Switch to docker run and re-enable tests * Add @nullable to AtlasDbConfigs * Set of minor modifications * Set of minor modifications * Annotate why we JsonIgnore onAcquireConnectionVisitor * Add tests for AtlasDbCliCommand#getOptionTypesForCommandMetadata * Refactor out addOptionToParser and add tests * Refactor out serialisation command * Fix-up checkstyle * Move timestamp file clean up into method * Add a new Leader configuration page * Make timeout 2 minutes * Fix checkstyle * Use correct jacoco version * Use assserThat from unit * Refactor CassandraSchemaLockTest and cassandra yml * Cleanup container logs * Cleanup build.gradle * Reduce timeout, extract CONTAINER_LOGS_DIRECTORY * minor doc nits * Prep atlasdb-dbkvs for baseline * Prepare atlasdb-dbkvs for findbugs * Enable baseline on atlasdb-dbkvs * Remove @after method * Increase timeout * Remove dependencies on ete1 as this is causing hanging builds. Fixed in #793 * Newline for Ben * Select the first leader according to alphabetical order to make sure lockCreator is deterministic. * Add tests * Fix CassandraConnectionTest by setting a valid quorumSize * Add docs change * Fix directory structure in atlasdb-impl-shared/src/test/java * Add profiling KVS wrapper to default kvs startup, use same order as internal product * Add necessary documentation for profiling kvs' * Docs nit: add comma before "which" * Add check to make sure it's possible to reach quorum * Wait for DB start * Prep atlasdb-impl-shared for baseline. Disable abbreviation checking in files with KV in their name. * Prep atlasdb-impl-shared tests for baseline * Get atlasdb-impl-shared and tests working with findbugs. Exclude DynamicPartitionMapImpl from synchronisation checks until later * Enable baseline for atlasdb-impl-shared * Code review fixes * Rename atlasdb-server -> atlasdb-service, atlasdb-server-dropwizard -> atlasdb-service-server * Add release notes * Prep atlasdb-cassandra for baseline * Fix-up some FindBugs for atlasdb-cassandra * Enable baseline on atlasdb-cassandra * Fix-up code review comments * ignore eclipse generated stuff * Reduce number of threads and timeout * Expand title line and fix link to file * Cleanup docs * Docs changes * Docs: Add link from Logging page to Enabling C* Tracing page. * Docs: minor fixes, mostly capitalisation * release notes edits for 0.12.0 * Rename link * Check for unique key constraints when inserting tables into metadata * Fix-up ETE dependencies so that we wait for the DB box to be created * another atlas -> atlasdb * Refactor DbkvsTestSuite so it runs more consistently locally * Remove test that checks duplicate table names throws. Cassandra doesn't throw, so we shouldn't really either * fix index.rst pages * edit breaking change note * final 0.12.0 release notes nits * Convert findbugs annotation version to a constant * Prep atlasdb-config for baseline * Enable baseline for atlasdb-config * Move release notes to right version * fix compile errors * publish test artifacts * add licenses * fix checkstyle errors * fixed errors from bad merge * fix checkstyle rule failures caused by merge * fix findbugs errors
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/airlift/airline/blob/0.7/src/main/java/io/airlift/airline/GlobalUsageSummary.java#L67-L69 converts hidden options to nulls.
That list is used just below that in
appendWordswhich throws a NPE.The text was updated successfully, but these errors were encountered: