kaspersorensen commented on issue #21: Upgrading MetaModel dependency from
5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-455048291
This passes now :-) I had to change the build so that the `Dockerfile` uses
a simply `COPY` operation. So now the
LosD commented on issue #21: Upgrading MetaModel dependency from 5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-455865891
It might be too big a change to make here, but it seems like Travis needs to
have caching explicitly enabled if we want to use
kaspersorensen commented on issue #21: Upgrading MetaModel dependency from
5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-454074417
Hmm I don't know what's going on. My last commit was to make the maven build
"quiet" inside the container, just to
LosD commented on issue #21: Upgrading MetaModel dependency from 5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-454086329
Quick builds are nice, but so is a simple build process, so the current way
might still be the best way, depending on how much is
kaspersorensen commented on issue #21: Upgrading MetaModel dependency from
5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-456239172
Yeah the problem is also just that the old Docker base image was just a
clean Maven install, so even just downloading
LosD commented on issue #21: Upgrading MetaModel dependency from 5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-456300600
Hehe, I _do_ think that in "proper" CI, you are only ever allowed to build
any releasable version _once_, so in that regard, this
kaspersorensen commented on issue #21: Upgrading MetaModel dependency from
5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-456881016
Yes let's merge this :-) Would be cool to see what you can do with Travis ;-)
kaspersorensen opened a new pull request #21: Upgrading MetaModel dependency
from 5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21
This is an automated message from the Apache Git Service.
To respond
LosD commented on issue #21: Upgrading MetaModel dependency from 5.1.0 to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-453959169
Seems like the Membrane server never responded.
BTW, that is quite a timeout on the client, 33.33 hours. If it's
LosD edited a comment on issue #21: Upgrading MetaModel dependency from 5.1.0
to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-453959169
Seems like the Membrane server never responded.
BTW, that is quite a timeout on the client, 33.33 hours. If it's
LosD edited a comment on issue #21: Upgrading MetaModel dependency from 5.1.0
to 5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21#issuecomment-453959169
Seems like the Membrane server never responded.
BTW, that is quite a timeout on the client, 33.33 hours. If it's
kaspersorensen commented on issue #217: More efficiently perform row count
operations for HBase.
URL: https://github.com/apache/metamodel/pull/217#issuecomment-478424839
LGTM
This is an automated message from the Apache Git
kaspersorensen commented on issue #213: METAMODEL-1172: Improves searching
based on an expression in MapValueFunction
URL: https://github.com/apache/metamodel/pull/213#issuecomment-475210929
The build failed on 1/4 of the JDKs that it's running on. Maybe just a flaky
test somewhere. You
kaspersorensen commented on issue #212: METAMODEL-1210: Module to read .arff
files
URL: https://github.com/apache/metamodel/pull/212#issuecomment-476033479
Thanks for checking it @LosD. Merging! :-)
This is an automated
asfgit merged pull request #212: METAMODEL-1210: Module to read .arff files
URL: https://github.com/apache/metamodel/pull/212
This is an automated message from the Apache Git Service.
To respond to the message, please log on
LosD commented on issue #212: METAMODEL-1210: Module to read .arff files
URL: https://github.com/apache/metamodel/pull/212#issuecomment-476002303
A short scan showed nothing worrying. It's rather self-contained, so I guess
we can make a quick merge.
Interesting file format, btw!
kaspersorensen commented on issue #217: More efficiently perform row count
operations for HBase.
URL: https://github.com/apache/metamodel/pull/217#issuecomment-478127289
Hi @q977734161 you can pull/merge from my branch now. I believe that should
stabilize your build.
asfgit merged pull request #215: Prepare 5.3.0 release
URL: https://github.com/apache/metamodel/pull/215
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
kaspersorensen commented on issue #217: More efficiently perform row count
operations for HBase.
URL: https://github.com/apache/metamodel/pull/217#issuecomment-478103945
Hmm @q977734161 I'm trying to fix the build/test issues/instability over in
#215. Once it passes, I suggest you merge
q977734161 commented on issue #217: More efficiently perform row count
operations for HBase.
URL: https://github.com/apache/metamodel/pull/217#issuecomment-478205038
I pull/merge from your branch and rebuild,now it's succuess.
q977734161 commented on issue #217: More efficiently perform row count
operations for HBase.
URL: https://github.com/apache/metamodel/pull/217#issuecomment-478004604
ok,I will rebuild.
This is an automated message from the
kaspersorensen commented on issue #217: More efficiently perform row count
operations for HBase.
URL: https://github.com/apache/metamodel/pull/217#issuecomment-477992414
Please try a rebuild.
Yuck this is getting to often that I say this. We need to get those unstable
tests fixed.
kaspersorensen commented on issue #214: Add a method of creating a table with
splitKey
URL: https://github.com/apache/metamodel/pull/214#issuecomment-477733526
LGTM!
This is an automated message from the Apache Git Service.
asfgit closed pull request #214: Add a method of creating a table with splitKey
URL: https://github.com/apache/metamodel/pull/214
This is an automated message from the Apache Git Service.
To respond to the message, please
kaspersorensen opened a new pull request #215: Prepare 5.3.0 release
URL: https://github.com/apache/metamodel/pull/215
Some final touches needed before I would stage a release for MetaModel v.
5.3.0.
This is an automated
q977734161 opened a new pull request #216: More efficiently perform row count
operations for HBase
URL: https://github.com/apache/metamodel/pull/216
A filter that will only return the key component of each KV (the value will
be rewritten as empty).
This filter can be used to grab all
q977734161 opened a new pull request #217: More efficiently perform row count
operations for HBase.
URL: https://github.com/apache/metamodel/pull/217
A filter that will only return the first KV from each row.This filter can be
used to more efficiently perform row count operations.
q977734161 closed pull request #216: More efficiently perform row count
operations for HBase
URL: https://github.com/apache/metamodel/pull/216
This is an automated message from the Apache Git Service.
To respond to the
q977734161 commented on a change in pull request #214: Add a method of creating
a table with splitKey
URL: https://github.com/apache/metamodel/pull/214#discussion_r269846086
##
File path: hbase/src/main/java/org/apache/metamodel/hbase/HBaseClient.java
##
@@ -151,26
asfgit merged pull request #217: More efficiently perform row count operations
for HBase.
URL: https://github.com/apache/metamodel/pull/217
This is an automated message from the Apache Git Service.
To respond to the
q977734161 opened a new pull request #218: Use Scan.addColumn(...) instead of
Scan.addFamily(...) when has qualifier.
URL: https://github.com/apache/metamodel/pull/218
when columns contain family and column name,using Scan.addColumn(...) method
to reduce network pressure and decrease
kaspersorensen commented on a change in pull request #209: Added the DynamoDB
module to the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r259604835
##
File path:
kaspersorensen commented on a change in pull request #209: Added the DynamoDB
module to the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r259604829
##
File path:
LosD commented on a change in pull request #209: Added the DynamoDB module to
the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r259616201
##
File path:
LosD commented on a change in pull request #209: Added the DynamoDB module to
the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r259616412
##
File path:
LosD commented on a change in pull request #209: Added the DynamoDB module to
the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r259616201
##
File path:
asfgit merged pull request #208: the result of column.getNativeType() may be
null and add pagination function to the Hive sql.
URL: https://github.com/apache/metamodel/pull/208
This is an automated message from the Apache
asfgit merged pull request #209: Added the DynamoDB module to the `full`
module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209
This is an automated message from the Apache Git
q977734161 opened a new pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210
Apache Impala support.Because this is no public repo for Impala jdbc,so I do
not add Impala dependency in pom.xml
q977734161 opened a new pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210
Apache Impala support.Because this is no public repo for Impala jdbc,so I do
not add Impala dependency in pom.xml
kaspersorensen commented on a change in pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210#discussion_r260371722
##
File path:
jdbc/src/main/java/org/apache/metamodel/jdbc/dialects/ImpalaQueryRewriter.java
##
@@ -0,0 +1,32 @@
+/**
+
kaspersorensen commented on issue #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210#issuecomment-469521278
LGTM. Merging :-)
This is an automated message from the Apache Git Service.
To respond to
asfgit merged pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
gkatiforis opened a new pull request #211: METAMODEL-1207: fix regex pattern in
order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211
Resolves METAMODEL-1207
The Regex pattern used to fetch the database major version is repleced with
the
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263098215
##
File path:
kaspersorensen commented on a change in pull request #208: the result of
column.getNativeType() may be null and add pagination function to the Hive sql.
URL: https://github.com/apache/metamodel/pull/208#discussion_r259643317
##
File path:
kaspersorensen commented on a change in pull request #208: the result of
column.getNativeType() may be null and add pagination function to the Hive sql.
URL: https://github.com/apache/metamodel/pull/208#discussion_r259643300
##
File path:
kaspersorensen commented on a change in pull request #208: the result of
column.getNativeType() may be null and add pagination function to the Hive sql.
URL: https://github.com/apache/metamodel/pull/208#discussion_r259643323
##
File path:
kaspersorensen commented on a change in pull request #208: the result of
column.getNativeType() may be null and add pagination function to the Hive sql.
URL: https://github.com/apache/metamodel/pull/208#discussion_r259643332
##
File path:
q977734161 commented on a change in pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210#discussion_r261857393
##
File path:
jdbc/src/main/java/org/apache/metamodel/jdbc/dialects/ImpalaQueryRewriter.java
##
@@ -0,0 +1,32 @@
+/**
+ *
q977734161 commented on a change in pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210#discussion_r260549095
##
File path:
jdbc/src/main/java/org/apache/metamodel/jdbc/dialects/ImpalaQueryRewriter.java
##
@@ -0,0 +1,32 @@
+/**
+ *
q977734161 commented on a change in pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210#discussion_r260549095
##
File path:
jdbc/src/main/java/org/apache/metamodel/jdbc/dialects/ImpalaQueryRewriter.java
##
@@ -0,0 +1,32 @@
+/**
+ *
q977734161 commented on a change in pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210#discussion_r260549095
##
File path:
jdbc/src/main/java/org/apache/metamodel/jdbc/dialects/ImpalaQueryRewriter.java
##
@@ -0,0 +1,32 @@
+/**
+ *
kaspersorensen commented on a change in pull request #210: Apache Impala support
URL: https://github.com/apache/metamodel/pull/210#discussion_r260550323
##
File path:
jdbc/src/main/java/org/apache/metamodel/jdbc/dialects/ImpalaQueryRewriter.java
##
@@ -0,0 +1,32 @@
+/**
+
kaspersorensen opened a new pull request #212: METAMODEL-1210: Module to read
.arff files
URL: https://github.com/apache/metamodel/pull/212
This is an automated message from the Apache Git Service.
To respond to the
gkatiforis commented on issue #211: METAMODEL-1207: fix regex pattern in order
to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#issuecomment-470418848
I could create a new VersionParser class and write the appropriate unit
tests. Which is the
kaspersorensen commented on issue #211: METAMODEL-1207: fix regex pattern in
order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#issuecomment-470422765
Probably just add it in the `org.apache.metamodel.jdbc` package and make the
class
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263957905
##
File path:
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263958384
##
File path:
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263957249
##
File path:
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263957415
##
File path:
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263958036
##
File path:
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263957828
##
File path:
kaspersorensen commented on a change in pull request #211: METAMODEL-1207: fix
regex pattern in order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#discussion_r263957996
##
File path:
asfgit closed pull request #211: METAMODEL-1207: fix regex pattern in order to
fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211
This is an automated message from the Apache Git
asfgit closed pull request #190: Added cache for prepared statements in
JdbcUpdateCallback
URL: https://github.com/apache/metamodel/pull/190
This is an automated message from the Apache Git Service.
To respond to the
asfgit closed pull request #156: Checkstyle integration
URL: https://github.com/apache/metamodel/pull/156
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
kaspersorensen commented on issue #211: METAMODEL-1207: fix regex pattern in
order to fetch the correct version in all cases
URL: https://github.com/apache/metamodel/pull/211#issuecomment-471190375
Looks good to me. One of the builds failed but it is unrelated (flaky Excel
module test).
gkatiforis opened a new pull request #213: METAMODEL-1172: Improves searching
based on an expression in MapValueFunction
URL: https://github.com/apache/metamodel/pull/213
Resolves METAMODEL-1172:
Adds support in cases that the outer most type is a list or array.
Adds support for
kaspersorensen commented on a change in pull request #214: Add a method of
creating a table with splitKey
URL: https://github.com/apache/metamodel/pull/214#discussion_r268305667
##
File path: hbase/src/main/java/org/apache/metamodel/hbase/HBaseClient.java
##
@@ -151,26
kaspersorensen commented on issue #213: METAMODEL-1172: Improves searching
based on an expression in MapValueFunction
URL: https://github.com/apache/metamodel/pull/213#issuecomment-475748506
Looks good to me.
This is an
kaspersorensen commented on issue #212: METAMODEL-1210: Module to read .arff
files
URL: https://github.com/apache/metamodel/pull/212#issuecomment-475749090
It's been a while and I could self-merge it based on "lazy concensus" but
I'd love it if someone took a quick glance at this.
asfgit merged pull request #213: METAMODEL-1172: Improves searching based on
an expression in MapValueFunction
URL: https://github.com/apache/metamodel/pull/213
This is an automated message from the Apache Git Service.
To
q977734161 opened a new pull request #214: Add a method of creating a table
with splitKey
URL: https://github.com/apache/metamodel/pull/214
Usually hbase has a lot of data to write, in order to spread the write
pressure, the pre-partition will be created in advance, so I add a method of
LosD commented on a change in pull request #209: Added the DynamoDB module to
the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r257452943
##
File path:
LosD commented on a change in pull request #209: Added the DynamoDB module to
the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r257452923
##
File path:
LosD commented on a change in pull request #209: Added the DynamoDB module to
the `full` module. And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#discussion_r257453237
##
File path:
LosD commented on issue #209: Added the DynamoDB module to the `full` module.
And created DataContextFactory for it.
URL: https://github.com/apache/metamodel/pull/209#issuecomment-464317991
Error seems unrelated (apparently more Excel instability)
asfgit merged pull request #21: Upgrading MetaModel dependency from 5.1.0 to
5.2.0
URL: https://github.com/apache/metamodel-membrane/pull/21
This is an automated message from the Apache Git Service.
To respond to the
q977734161 commented on issue #218: Use Scan.addColumn(...) instead of
Scan.addFamily(...) when has qualifier.
URL: https://github.com/apache/metamodel/pull/218#issuecomment-481085627
ok,I will commit another empty commit to trigger rebuild.
kaspersorensen commented on issue #218: Use Scan.addColumn(...) instead of
Scan.addFamily(...) when has qualifier.
URL: https://github.com/apache/metamodel/pull/218#issuecomment-481085304
Sorry, seems the test flakyness is not completely over. Can you commit
another empty commit to
kaspersorensen commented on a change in pull request #218: Use
Scan.addColumn(...) instead of Scan.addFamily(...) when has qualifier.
URL: https://github.com/apache/metamodel/pull/218#discussion_r273257498
##
File path:
kaspersorensen commented on a change in pull request #218: Use
Scan.addColumn(...) instead of Scan.addFamily(...) when has qualifier.
URL: https://github.com/apache/metamodel/pull/218#discussion_r273257643
##
File path:
q977734161 commented on a change in pull request #218: Use Scan.addColumn(...)
instead of Scan.addFamily(...) when has qualifier.
URL: https://github.com/apache/metamodel/pull/218#discussion_r273284050
##
File path:
kaspersorensen opened a new pull request #219: Improved stability of Salesforce
test and exception handling.
URL: https://github.com/apache/metamodel/pull/219
This should avoid failures such as the one that broke the assertion in
Travis CI recently;
kaspersorensen commented on issue #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#issuecomment-498811206
LGTM. I'll fix the few remaining cosmetic issues and push.
This is an automated
kaspersorensen commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r290460236
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/WebServer.java
##
@@ -85,7 +88,16 @@
kaspersorensen commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r290460161
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/WebServer.java
##
@@ -85,7 +88,16 @@
kaspersorensen commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r289007111
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/WebServer.java
##
@@ -85,7 +88,15 @@
kaspersorensen commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r289007032
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/WebServer.java
##
@@ -52,10 +53,12 @@
kaspersorensen commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r289007231
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/CorsHandlers.java
##
@@ -0,0 +1,28 @@
kaspersorensen commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r289006546
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/WebServer.java
##
@@ -6,9 +6,9 @@
*
nvitucci commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r289038577
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/CorsHandlers.java
##
@@ -0,0 +1,28 @@
+/**
+
nvitucci commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r289038453
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/WebServer.java
##
@@ -52,10 +53,12 @@ public
nvitucci commented on a change in pull request #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#discussion_r289037532
##
File path:
undertow/src/main/java/org/apache/metamodel/membrane/server/WebServer.java
##
@@ -6,9 +6,9 @@
* to you
nvitucci commented on issue #22: Add support for CORS
URL: https://github.com/apache/metamodel-membrane/pull/22#issuecomment-498315847
Anything else needed?
This is an automated message from the Apache Git Service.
To respond
kaspersorensen commented on a change in pull request #221: Access CSVParser in
a synchronized manner
URL: https://github.com/apache/metamodel/pull/221#discussion_r293771783
##
File path: csv/src/main/java/org/apache/metamodel/csv/CsvConfiguration.java
##
@@ -195,4 +199,17
kaspersorensen commented on a change in pull request #221: Access CSVParser in
a synchronized manner
URL: https://github.com/apache/metamodel/pull/221#discussion_r293772108
##
File path: csv/src/main/java/org/apache/metamodel/csv/SingleLineCsvDataSet.java
##
@@ -44,20
LosD commented on a change in pull request #221: Access CSVParser in a
synchronized manner
URL: https://github.com/apache/metamodel/pull/221#discussion_r293777107
##
File path: csv/src/main/java/org/apache/metamodel/csv/CsvConfiguration.java
##
@@ -195,4 +199,17 @@ public
LosD commented on a change in pull request #221: Access CSVParser in a
synchronized manner
URL: https://github.com/apache/metamodel/pull/221#discussion_r293782781
##
File path: csv/src/main/java/org/apache/metamodel/csv/CsvConfiguration.java
##
@@ -195,4 +199,17 @@ public
1 - 100 of 495 matches
Mail list logo