[jira] [Commented] (HIVE-12927) HBase metastore: sequences are not safe
[ https://issues.apache.org/jira/browse/HIVE-12927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15134617#comment-15134617 ] Hive QA commented on HIVE-12927: Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12786166/HIVE-12927.patch {color:green}SUCCESS:{color} +1 due to 1 test(s) being added or modified. {color:red}ERROR:{color} -1 due to 9 failed/errored test(s), 10052 tests executed *Failed tests:* {noformat} org.apache.hadoop.hive.cli.TestMiniTezCliDriver.org.apache.hadoop.hive.cli.TestMiniTezCliDriver org.apache.hadoop.hive.cli.TestMiniTezCliDriver.testCliDriver_schema_evol_orc_vec_mapwork_table org.apache.hadoop.hive.cli.TestNegativeCliDriver.testNegativeCliDriver_authorization_uri_import org.apache.hadoop.hive.metastore.TestHiveMetaStorePartitionSpecs.testAddPartitions org.apache.hadoop.hive.metastore.TestHiveMetaStorePartitionSpecs.testFetchingPartitionsWithDifferentSchemas org.apache.hadoop.hive.metastore.TestHiveMetaStorePartitionSpecs.testGetPartitionSpecs_WithAndWithoutPartitionGrouping org.apache.hadoop.hive.thrift.TestHadoopAuthBridge23.testMetastoreProxyUser org.apache.hadoop.hive.thrift.TestHadoopAuthBridge23.testSaslWithHiveMetaStore org.apache.hive.jdbc.TestSSL.testSSLVersion {noformat} Test results: http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/6876/testReport Console output: http://ec2-174-129-184-35.compute-1.amazonaws.com/jenkins/job/PreCommit-HIVE-TRUNK-Build/6876/console Test logs: http://ec2-174-129-184-35.compute-1.amazonaws.com/logs/PreCommit-HIVE-TRUNK-Build-6876/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase Tests exited with: TestsFailedException: 9 tests failed {noformat} This message is automatically generated. ATTACHMENT ID: 12786166 - PreCommit-HIVE-TRUNK-Build > HBase metastore: sequences are not safe > --- > > Key: HIVE-12927 > URL: https://issues.apache.org/jira/browse/HIVE-12927 > Project: Hive > Issue Type: Bug >Affects Versions: 2.0.0 >Reporter: Sergey Shelukhin >Assignee: Alan Gates >Priority: Critical > Attachments: HIVE-12927.patch > > > {noformat} > long getNextSequence(byte[] sequence) throws IOException { > {noformat} > Is not safe in presence of any concurrency. It should use HBase increment API. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HIVE-12927) HBase metastore: sequences are not safe
[ https://issues.apache.org/jira/browse/HIVE-12927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15133294#comment-15133294 ] Alan Gates commented on HIVE-12927: --- See comments on HIVE-12892 on why increment won't work in that case. In this case where I'm explicitly circumventing the transaction management it would be viable. Is it better than checkAndPut? I don't care which we use here. > HBase metastore: sequences are not safe > --- > > Key: HIVE-12927 > URL: https://issues.apache.org/jira/browse/HIVE-12927 > Project: Hive > Issue Type: Bug >Affects Versions: 2.0.0 >Reporter: Sergey Shelukhin >Assignee: Alan Gates >Priority: Critical > Attachments: HIVE-12927.patch > > > {noformat} > long getNextSequence(byte[] sequence) throws IOException { > {noformat} > Is not safe in presence of any concurrency. It should use HBase increment API. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HIVE-12927) HBase metastore: sequences are not safe
[ https://issues.apache.org/jira/browse/HIVE-12927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15131586#comment-15131586 ] Sergey Shelukhin commented on HIVE-12927: - Hmm. Can increment API be used in the txn managers? that is separate from checkAndPut. See the patch at HBASE-12892 > HBase metastore: sequences are not safe > --- > > Key: HIVE-12927 > URL: https://issues.apache.org/jira/browse/HIVE-12927 > Project: Hive > Issue Type: Bug >Affects Versions: 2.0.0 >Reporter: Sergey Shelukhin >Assignee: Alan Gates >Priority: Critical > Attachments: HIVE-12927.patch > > > {noformat} > long getNextSequence(byte[] sequence) throws IOException { > {noformat} > Is not safe in presence of any concurrency. It should use HBase increment API. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HIVE-12927) HBase metastore: sequences are not safe
[ https://issues.apache.org/jira/browse/HIVE-12927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15124486#comment-15124486 ] Alan Gates commented on HIVE-12927: --- My original plan for this was to make use the transaction server that we'll need to add for HBase metastore anyway (most likely Omid). But given that we see a lot of contention on the sequences in the RDBMS world in ACID it might make sense to rely on HBase simpler check and put rather than the transaction service here. > HBase metastore: sequences are not safe > --- > > Key: HIVE-12927 > URL: https://issues.apache.org/jira/browse/HIVE-12927 > Project: Hive > Issue Type: Bug >Reporter: Sergey Shelukhin >Priority: Critical > > {noformat} > long getNextSequence(byte[] sequence) throws IOException { > {noformat} > Is not safe in presence of any concurrency. It should use HBase increment API. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HIVE-12927) HBase metastore: sequences are not safe
[ https://issues.apache.org/jira/browse/HIVE-12927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15116529#comment-15116529 ] Sergey Shelukhin commented on HIVE-12927: - [~alangates] [~daijy] fyi > HBase metastore: sequences are not safe > --- > > Key: HIVE-12927 > URL: https://issues.apache.org/jira/browse/HIVE-12927 > Project: Hive > Issue Type: Bug >Reporter: Sergey Shelukhin > > {noformat} > long getNextSequence(byte[] sequence) throws IOException { > {noformat} > Is not safe in presence of any concurrency. It should use HBase increment API. -- This message was sent by Atlassian JIRA (v6.3.4#6332)