Updated readme to include troubleshooting info #47

joshzarrabi · 2015-06-02T23:06:52Z

More code for the phone, added two new classes, not really sure how to implement them with the cordova but thats all thats remaining I think...

AmplabJenkins · 2015-06-02T23:10:03Z

Merged build triggered.

AmplabJenkins · 2015-06-02T23:10:10Z

Merged build started.

AmplabJenkins · 2015-06-02T23:11:01Z

Merged build finished. Test PASSed.

AmplabJenkins · 2015-06-05T11:35:05Z

Build triggered.

AmplabJenkins · 2015-06-05T11:35:10Z

Build started.

AmplabJenkins · 2015-06-05T11:35:29Z

Build finished. Test FAILed.

shankari · 2015-06-05T12:04:00Z

Two high level comments:

this is not using cordova! I see from your top level comment that you still need to figure that out. I would suggest looking at that first. Lack of support for cordova is a non-starter.
are you sure that it works offline - it seems to be an online sync mechanism at this point, when you wrap the URL into a "service client". A quick test for offline access/true background sync - turn off wifi on your laptop and then see if the screen still works in the emulator.
recall prof. culler's comment:

The real lesson is that even if you build for one, you try to create an abstraction layer so you have a little shim and as narrow an exposure as possible. Then the possibility of retargeting or multi-target is not so bad. Tends to focus the mind on what is needed, rather than what is provided. And yes, failure to do this in the meteor space doomed the result.

I am not sure that is happening right now - this is really Azure specific. That is not surprising, since you are only implementing Azure at this point. But it would be awesome to build this in right now as we go along.

Let me put in a concrete goal. You are going to implement 3 options, right? Can we select between the three in the settings UI? Having that as a concrete goal will help ensure that we build a proper abstraction layer, rather than tying it together really tightly.

joshzarrabi · 2015-06-05T18:57:18Z

Hi!

In order for this to be native I'm almost 100% sure it needs to be in
java, the Cordova can call it.
Offline stuff comes later in the tutorial.
Yeah I will do that

Thanks for the feedback!

Josh

On Friday, June 5, 2015, shankari [email protected] wrote:

Two high level comments:

this is not using cordova! I see from your top level comment that
you still need to figure that out. I would suggest looking at that first.
Lack of support for cordova is a non-starter.

are you sure that it works offline - it seems to be an online sync
mechanism at this point, when you wrap the URL into a "service client". A
quick test for offline access/true background sync - turn off wifi on your
laptop and then see if the screen still works in the emulator.

recall prof. culler's comment:

The real lesson is that even if you build for one, you try to create an
abstraction layer so you have a little shim and as narrow an exposure as
possible. Then the possibility of retargeting or multi-target is not so
bad. Tends to focus the mind on what is needed, rather than what is
provided. And yes, failure to do this in the meteor space doomed the result.

I am not sure that is happening right now - this is really Azure specific.
That is not surprising, since you are only implementing Azure at this
point. But it would be awesome to build this in right now as we go along.

Let me put in a concrete goal. You are going to implement 3 options,
right? Can we select between the three in the settings UI? Having that as a
concrete goal will help ensure that we build a proper abstraction layer,
rather than tying it together really tightly.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109269169
.

shankari · 2015-06-05T19:05:22Z

Let me clarify further. We want to do the sync in native code, but then we want to read the synced data in cordova. How can cordova call it? Is there a cordova plugin? I would start with checking to see if there is a cordova plugin because if there isn't, it is a big downside for us and we can use that as a filter function. That's why I suggested adding "cordova support" as an entry into the comparison table.

shankari · 2015-06-05T19:07:26Z

it is a big downside for us and we can use that as a filter function

ie, we should only evaluate projects that have cordova support. ie.e, we should filter out projects that don't have cordova support, similar to the way we filtered out javascript-only solutions like meteor.

joshzarrabi · 2015-06-05T19:09:33Z

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it today to
see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected] wrote:

Let me clarify further. We want to do the sync in native code, but
then we want to read the synced data in cordova. How can cordova call it?
Is there a cordova plugin? I would start with checking to see if there is a
cordova plugin because if there isn't, it is a big downside for us and we
can use that as a filter function. That's why I suggested adding "cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120
.

joshzarrabi · 2015-06-05T21:03:28Z

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the phone, then
we use a plug in from cordova to grab it from there when we want to load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi [email protected]
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected] wrote:

Let me clarify further. We want to do the sync in native code, but
then we want to read the synced data in cordova. How can cordova call it?
Is there a cordova plugin? I would start with checking to see if there is a
cordova plugin because if there isn't, it is a big downside for us and we
can use that as a filter function. That's why I suggested adding "cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120
.

shankari · 2015-06-05T22:20:49Z

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the www/dbHelper.js in the
Cordova code.

What does "we need to store the data" mean? Don't they already do that? If
we have to deal with storage read/write and scheduling the sync then there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" [email protected] wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the phone, then
we use a plug in from cordova to grab it from there when we want to load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected] wrote:

Let me clarify further. We want to do the sync in native code, but
then we want to read the synced data in cordova. How can cordova call
it?
Is there a cordova plugin? I would start with checking to see if there
is a
cordova plugin because if there isn't, it is a big downside for us and
we
can use that as a filter function. That's why I suggested adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048
.

joshzarrabi · 2015-06-05T23:08:41Z

Yeah, as I'm coding this I'm not sure what advantage we would really get
using this solution...

especially if what we have already works.
Because it is basically just doing what we already have...

It would probably be more elegant, but I'm not sure how much better it
would actually end up being.

On Fri, Jun 5, 2015 at 3:20 PM, shankari [email protected] wrote:

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the www/dbHelper.js in the
Cordova code.

What does "we need to store the data" mean? Don't they already do that? If
we have to deal with storage read/write and scheduling the sync then there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" [email protected] wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the phone,
then
we use a plug in from cordova to grab it from there when we want to load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected] wrote:

Let me clarify further. We want to do the sync in native code, but
then we want to read the synced data in cordova. How can cordova call
it?
Is there a cordova plugin? I would start with checking to see if there
is a
cordova plugin because if there isn't, it is a big downside for us and
we
can use that as a filter function. That's why I suggested adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109460056
.

joshzarrabi · 2015-06-05T23:24:01Z

I guess a good thing is that this helps us handle conflicts better ...

if (ex != null) {
// A conflict was detected; let's force the server to "win"
// by discarding the client version of the item
// Other policies could be used, such as prompt the user for
// which version to maintain.
JsonObject serverItem = ex.getValue();

if (serverItem == null) {
    // Item not returned in the exception, retrieving it from the server
    try {
        serverItem =

mClient.getTable(operation.getTableName()).lookUp(operation.getItemId()).get();
} catch (Exception e) {
throw new MobileServiceSyncHandlerException(e);
}
}

result = serverItem;

}

This is conveniently in the code

On Fri, Jun 5, 2015 at 4:08 PM, Joshua N Zarrabi [email protected]
wrote:

Yeah, as I'm coding this I'm not sure what advantage we would really get
using this solution...

especially if what we have already works.
Because it is basically just doing what we already have...

It would probably be more elegant, but I'm not sure how much better it
would actually end up being.

On Fri, Jun 5, 2015 at 3:20 PM, shankari [email protected] wrote:

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the www/dbHelper.js in
the
Cordova code.

What does "we need to store the data" mean? Don't they already do that? If
we have to deal with storage read/write and scheduling the sync then there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" [email protected] wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the phone,
then
we use a plug in from cordova to grab it from there when we want to load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it
today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected] wrote:

Let me clarify further. We want to do the sync in native code, but
then we want to read the synced data in cordova. How can cordova call
it?
Is there a cordova plugin? I would start with checking to see if
there
is a
cordova plugin because if there isn't, it is a big downside for us
and
we
can use that as a filter function. That's why I suggested adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109460056
.

shankari · 2015-06-05T23:36:27Z

It is not clear that this conflict resolution is the right thing to do,
specially since the user is entering information on the client; in other
words, the conflict resolution is convenient but wrong. Do they actually
schedule the true background synch? I think couchbase mobile may have
better support.
On Jun 5, 2015 4:24 PM, "Josh Zarrabi" [email protected] wrote:

I guess a good thing is that this helps us handle conflicts better ...

if (ex != null) {
// A conflict was detected; let's force the server to "win"
// by discarding the client version of the item
// Other policies could be used, such as prompt the user for
// which version to maintain.
JsonObject serverItem = ex.getValue();

if (serverItem == null) {
// Item not returned in the exception, retrieving it from the server
try {
serverItem =

mClient.getTable(operation.getTableName()).lookUp(operation.getItemId()).get();
} catch (Exception e) {
throw new MobileServiceSyncHandlerException(e);
}
}

result = serverItem;
}

This is conveniently in the code

On Fri, Jun 5, 2015 at 4:08 PM, Joshua N Zarrabi <[email protected]

wrote:

Yeah, as I'm coding this I'm not sure what advantage we would really get
using this solution...

especially if what we have already works.
Because it is basically just doing what we already have...

It would probably be more elegant, but I'm not sure how much better it
would actually end up being.

On Fri, Jun 5, 2015 at 3:20 PM, shankari [email protected]
wrote:

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the www/dbHelper.js in
the
Cordova code.

What does "we need to store the data" mean? Don't they already do that?
If
we have to deal with storage read/write and scheduling the sync then
there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" [email protected]
wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the phone,
then
we use a plug in from cordova to grab it from there when we want to
load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it
today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected] wrote:

Let me clarify further. We want to do the sync in native code,
but
then we want to read the synced data in cordova. How can cordova
call
it?
Is there a cordova plugin? I would start with checking to see if
there
is a
cordova plugin because if there isn't, it is a big downside for us
and
we
can use that as a filter function. That's why I suggested adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109460056

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109480272
.

joshzarrabi · 2015-06-06T00:04:18Z

I think you might be right, we may have to use push notification.

In all honesty azure didnt provide that great of documentation for Android
either.

Oh well...

Hopefully couchbase will provide better support.

On Fri, Jun 5, 2015 at 4:36 PM, shankari [email protected] wrote:

It is not clear that this conflict resolution is the right thing to do,
specially since the user is entering information on the client; in other
words, the conflict resolution is convenient but wrong. Do they actually
schedule the true background synch? I think couchbase mobile may have
better support.

On Jun 5, 2015 4:24 PM, "Josh Zarrabi" [email protected] wrote:

I guess a good thing is that this helps us handle conflicts better ...

if (ex != null) {
// A conflict was detected; let's force the server to "win"
// by discarding the client version of the item
// Other policies could be used, such as prompt the user for
// which version to maintain.
JsonObject serverItem = ex.getValue();

if (serverItem == null) {
// Item not returned in the exception, retrieving it from the server
try {
serverItem =

mClient.getTable(operation.getTableName()).lookUp(operation.getItemId()).get();
} catch (Exception e) {
throw new MobileServiceSyncHandlerException(e);
}
}

result = serverItem;
}

This is conveniently in the code

On Fri, Jun 5, 2015 at 4:08 PM, Joshua N Zarrabi <
[email protected]

wrote:

Yeah, as I'm coding this I'm not sure what advantage we would really
get
using this solution...

especially if what we have already works.
Because it is basically just doing what we already have...

It would probably be more elegant, but I'm not sure how much better it
would actually end up being.

On Fri, Jun 5, 2015 at 3:20 PM, shankari [email protected]
wrote:

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the www/dbHelper.js
in
the
Cordova code.

What does "we need to store the data" mean? Don't they already do
that?
If
we have to deal with storage read/write and scheduling the sync then
there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" [email protected]
wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the
phone,
then
we use a plug in from cordova to grab it from there when we want to
load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it
today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected]
wrote:

Let me clarify further. We want to do the sync in native code,
but
then we want to read the synced data in cordova. How can cordova
call
it?
Is there a cordova plugin? I would start with checking to see if
there
is a
cordova plugin because if there isn't, it is a big downside for
us
and
we
can use that as a filter function. That's why I suggested adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109460056

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109480272

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109483095
.

joshzarrabi · 2015-06-06T00:08:47Z

I think we would have to use push notifications to do what we want

On Fri, Jun 5, 2015 at 5:03 PM, Joshua N Zarrabi [email protected]
wrote:

I think you might be right, we may have to use push notification.

In all honesty azure didnt provide that great of documentation for Android
either.

Oh well...

Hopefully couchbase will provide better support.

On Fri, Jun 5, 2015 at 4:36 PM, shankari [email protected] wrote:

It is not clear that this conflict resolution is the right thing to do,
specially since the user is entering information on the client; in other
words, the conflict resolution is convenient but wrong. Do they actually
schedule the true background synch? I think couchbase mobile may have
better support.

On Jun 5, 2015 4:24 PM, "Josh Zarrabi" [email protected] wrote:

I guess a good thing is that this helps us handle conflicts better ...

if (ex != null) {
// A conflict was detected; let's force the server to "win"
// by discarding the client version of the item
// Other policies could be used, such as prompt the user for
// which version to maintain.
JsonObject serverItem = ex.getValue();

if (serverItem == null) {
// Item not returned in the exception, retrieving it from the server
try {
serverItem =

mClient.getTable(operation.getTableName()).lookUp(operation.getItemId()).get();
} catch (Exception e) {
throw new MobileServiceSyncHandlerException(e);
}
}

result = serverItem;
}

This is conveniently in the code

On Fri, Jun 5, 2015 at 4:08 PM, Joshua N Zarrabi <
[email protected]

wrote:

Yeah, as I'm coding this I'm not sure what advantage we would really
get
using this solution...

especially if what we have already works.
Because it is basically just doing what we already have...

It would probably be more elegant, but I'm not sure how much better it
would actually end up being.

On Fri, Jun 5, 2015 at 3:20 PM, shankari [email protected]
wrote:

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the www/dbHelper.js
in
the
Cordova code.

What does "we need to store the data" mean? Don't they already do
that?
If
we have to deal with storage read/write and scheduling the sync then
there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" [email protected]
wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the
phone,
then
we use a plug in from cordova to grab it from there when we want to
load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at it
today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected]
wrote:

Let me clarify further. We want to do the sync in native
code,
but
then we want to read the synced data in cordova. How can cordova
call
it?
Is there a cordova plugin? I would start with checking to see if
there
is a
cordova plugin because if there isn't, it is a big downside for
us
and
we
can use that as a filter function. That's why I suggested adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109460056

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109480272

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109483095
.

shankari · 2015-06-06T00:24:33Z

We don't actually use push notifications on android even for our solution.
I know it sounds cool but we don't use it.

What is their story for true background sync? Do they even support it, or
is it primarily oriented towards refreshing the UI display?
On Jun 5, 2015 5:04 PM, "Josh Zarrabi" [email protected] wrote:

I think you might be right, we may have to use push notification.

In all honesty azure didnt provide that great of documentation for Android
either.

Oh well...

Hopefully couchbase will provide better support.

On Fri, Jun 5, 2015 at 4:36 PM, shankari [email protected] wrote:

It is not clear that this conflict resolution is the right thing to do,
specially since the user is entering information on the client; in other
words, the conflict resolution is convenient but wrong. Do they actually
schedule the true background synch? I think couchbase mobile may have
better support.

On Jun 5, 2015 4:24 PM, "Josh Zarrabi" [email protected] wrote:

I guess a good thing is that this helps us handle conflicts better ...

if (ex != null) {
// A conflict was detected; let's force the server to "win"
// by discarding the client version of the item
// Other policies could be used, such as prompt the user for
// which version to maintain.
JsonObject serverItem = ex.getValue();

if (serverItem == null) {
// Item not returned in the exception, retrieving it from the server
try {
serverItem =

mClient.getTable(operation.getTableName()).lookUp(operation.getItemId()).get();

} catch (Exception e) {
throw new MobileServiceSyncHandlerException(e);
}
}

result = serverItem;
}

This is conveniently in the code

On Fri, Jun 5, 2015 at 4:08 PM, Joshua N Zarrabi <
[email protected]

wrote:

Yeah, as I'm coding this I'm not sure what advantage we would really
get
using this solution...

especially if what we have already works.
Because it is basically just doing what we already have...

It would probably be more elegant, but I'm not sure how much better
it
would actually end up being.

On Fri, Jun 5, 2015 at 3:20 PM, shankari [email protected]
wrote:

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the www/dbHelper.js
in
the
Cordova code.

What does "we need to store the data" mean? Don't they already do
that?
If
we have to deal with storage read/write and scheduling the sync then
there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" [email protected]
wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the
phone,
then
we use a plug in from cordova to grab it from there when we want
to
load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at
it
today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari [email protected]
wrote:

Let me clarify further. We want to do the sync in native
code,
but
then we want to read the synced data in cordova. How can
cordova
call
it?
Is there a cordova plugin? I would start with checking to see
if
there
is a
cordova plugin because if there isn't, it is a big downside for
us
and
we
can use that as a filter function. That's why I suggested
adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109460056

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109480272

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109483095

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109486444
.

joshzarrabi · 2015-06-06T00:27:34Z

I'm have trouble figuring it out. It seems like it's only for the ui, as in
it only checks for updates when you change something on the phone.

I feel like it should offer what we want, but the many hours I've spent
searching have been fruitless

On Friday, June 5, 2015, shankari [email protected] wrote:

We don't actually use push notifications on android even for our solution.
I know it sounds cool but we don't use it.

What is their story for true background sync? Do they even support it, or
is it primarily oriented towards refreshing the UI display?
On Jun 5, 2015 5:04 PM, "Josh Zarrabi" <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');> wrote:

I think you might be right, we may have to use push notification.

In all honesty azure didnt provide that great of documentation for
Android
either.

Oh well...

Hopefully couchbase will provide better support.

On Fri, Jun 5, 2015 at 4:36 PM, shankari <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');> wrote:

It is not clear that this conflict resolution is the right thing to do,
specially since the user is entering information on the client; in
other
words, the conflict resolution is convenient but wrong. Do they
actually
schedule the true background synch? I think couchbase mobile may have
better support.

On Jun 5, 2015 4:24 PM, "Josh Zarrabi" <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');> wrote:

I guess a good thing is that this helps us handle conflicts better
...

if (ex != null) {
// A conflict was detected; let's force the server to "win"
// by discarding the client version of the item
// Other policies could be used, such as prompt the user for
// which version to maintain.
JsonObject serverItem = ex.getValue();

if (serverItem == null) {
// Item not returned in the exception, retrieving it from the server
try {
serverItem =

mClient.getTable(operation.getTableName()).lookUp(operation.getItemId()).get();

} catch (Exception e) {
throw new MobileServiceSyncHandlerException(e);
}
}

result = serverItem;
}

This is conveniently in the code

On Fri, Jun 5, 2015 at 4:08 PM, Joshua N Zarrabi <
[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');

wrote:

Yeah, as I'm coding this I'm not sure what advantage we would
really
get
using this solution...

especially if what we have already works.
Because it is basically just doing what we already have...

It would probably be more elegant, but I'm not sure how much better
it
would actually end up being.

On Fri, Jun 5, 2015 at 3:20 PM, shankari <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');>
wrote:

FYI, that is exactly what my current solution does. Look at the
ModeClassificationHelper in the android code and the
www/dbHelper.js
in
the
Cordova code.

What does "we need to store the data" mean? Don't they already do
that?
If
we have to deal with storage read/write and scheduling the sync
then
there
is no huge advantage to a paid solution.

Shankari
On Jun 5, 2015 2:03 PM, "Josh Zarrabi" <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');>
wrote:

Ok, So I think I know what to do..

We use the native code to store the data in a SQLlite db on the
phone,
then
we use a plug in from cordova to grab it from there when we want
to
load
the page

On Fri, Jun 5, 2015 at 12:09 PM, Joshua N Zarrabi <
[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');>
wrote:

I saw something about inplementing it with Cordova ...

I didn't really look into it much but it's there. I'll look at
it
today
to see how to call it from Cordova

On Friday, June 5, 2015, shankari <[email protected]
javascript:_e(%7B%7D,'cvml','[email protected]');>
wrote:

Let me clarify further. We want to do the sync in native
code,
but
then we want to read the synced data in cordova. How can
cordova
call
it?
Is there a cordova plugin? I would start with checking to see
if
there
is a
cordova plugin because if there isn't, it is a big downside
for
us
and
we
can use that as a filter function. That's why I suggested
adding
"cordova
support" as an entry into the comparison table.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109400120

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109439048

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109460056

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109480272

.

—
Reply to this email directly or view it on GitHub
<

https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109483095

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109486444

.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109488233
.

shankari · 2015-06-06T01:40:08Z

I think that you should budget a fixed amount of time to look at this (x hours total) and move on to evaluating a different solution after that point.

After all, some of the advantages of moving to a paid solution are supposed to be ease of use and superior documentation. If a motivated UC Berkeley student can't figure out how to do something in x hours, then it is not easy and not well documented. So document the things that you tried, and move on to something else.

Remember that we always have our own solution as backup.

joshzarrabi · 2015-06-06T02:35:21Z

Okay!

Thanks Shankari!!

Josh

On Fri, Jun 5, 2015 at 6:40 PM, shankari [email protected] wrote:

I think that you should budget a fixed amount of time to look at this (x
hours total) and move on to evaluating a different solution after that
point.

After all, some of the advantages of moving to a paid solution are
supposed to be ease of use and superior documentation. If a motivated UC
Berkeley student can't figure out how to do something in x hours, then it
is not easy and not well documented. So document the things that you tried,
and move on to something else.

Remember that we always have our own solution as backup.

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-109496616
.

AmplabJenkins · 2015-06-10T06:55:03Z

Build triggered.

AmplabJenkins · 2015-06-10T06:55:08Z

Build started.

AmplabJenkins · 2015-06-10T06:55:35Z

Build finished. Test FAILed.

AmplabJenkins · 2015-06-11T06:00:04Z

Build triggered.

AmplabJenkins · 2015-06-11T23:20:28Z

Build finished. Test FAILed.

AmplabJenkins · 2015-06-16T22:50:03Z

Build triggered.

AmplabJenkins · 2015-06-16T22:50:13Z

Build started.

AmplabJenkins · 2015-06-16T22:50:26Z

Build finished. Test FAILed.

AmplabJenkins · 2015-06-18T01:20:04Z

Build triggered.

AmplabJenkins · 2015-06-18T01:20:13Z

Build started.

AmplabJenkins · 2015-06-18T01:20:25Z

Build finished. Test FAILed.

shankari · 2015-06-18T06:36:32Z

android/app/src/main/java/edu/berkeley/eecs/e_mission/SettingsActivity.java

+        try {
+            String chosenBackend = String.valueOf(parent.getItemAtPosition(pos).toString());
+            Log.d(TAG, "Chosen Backend is " + chosenBackend);
+            ChosenBackend = chosenBackend;


This is not a valid solution because the activity can be killed at any time by the system to reduce memory pressure. You need to store this somewhere. SharedPreferences is a reasonable solution.

AmplabJenkins · 2015-06-19T00:10:04Z

Build triggered.

AmplabJenkins · 2015-06-19T00:10:12Z

Build started.

AmplabJenkins · 2015-06-19T00:10:30Z

Build finished. Test FAILed.

shankari · 2015-06-19T02:37:20Z

android/app/src/main/java/edu/berkeley/eecs/e_mission/data_sync/ConfirmTripsAdapter.java

+				e.printStackTrace();
+			}
+		}
+			if (syncSkip == true) {


I would assume that the sanity checks need to be the first things in the sync.

shankari · 2015-06-19T03:00:16Z

Great to see that this is working!

joshzarrabi · 2015-06-19T18:16:13Z

Hi SHankari,

Thanks for the review ...
The code is still really rough and full of my random debugging and
experimenting ...

I'll have to clean it up after I get authentication working

On Thu, Jun 18, 2015 at 8:00 PM, shankari [email protected] wrote:

Great to see that this is working!

—
Reply to this email directly or view it on GitHub
https://github.com/e-mission/e-mission-phone/pull/47#issuecomment-113352102
.

AmplabJenkins · 2015-06-24T17:50:04Z

Build triggered.

AmplabJenkins · 2015-06-24T17:50:13Z

Build started.

AmplabJenkins · 2015-06-24T17:50:27Z

Build finished. Test FAILed.

Updated readme to include troubleshooting info

86c26a1

First changes for Azure

1d2564d

added support for windows, more sync code

b40489c

Starting to work on authentication,.. not working yet

f247569

joshzarrabi added 2 commits June 15, 2015 15:43

Starting the abstraction of background sync

101d5bc

More updates for couchbase and azure abstraction

8e234b1

Couchbase working

8189c1b

shankari reviewed Jun 18, 2015
View reviewed changes

Syncing on couchbase works

4fb05aa

shankari reviewed Jun 19, 2015
View reviewed changes

Couchbase good sans any secure authentication

6a4318b

Updated readme to include troubleshooting info #47

Are you sure you want to change the base?

Updated readme to include troubleshooting info #47

Conversation

joshzarrabi commented Jun 2, 2015

AmplabJenkins commented Jun 2, 2015

AmplabJenkins commented Jun 2, 2015

AmplabJenkins commented Jun 2, 2015

AmplabJenkins commented Jun 5, 2015

AmplabJenkins commented Jun 5, 2015

AmplabJenkins commented Jun 5, 2015

shankari commented Jun 5, 2015

joshzarrabi commented Jun 5, 2015

shankari commented Jun 5, 2015

shankari commented Jun 5, 2015

joshzarrabi commented Jun 5, 2015

joshzarrabi commented Jun 5, 2015

shankari commented Jun 5, 2015

joshzarrabi commented Jun 5, 2015

joshzarrabi commented Jun 5, 2015

shankari commented Jun 5, 2015

joshzarrabi commented Jun 6, 2015

joshzarrabi commented Jun 6, 2015

shankari commented Jun 6, 2015

joshzarrabi commented Jun 6, 2015

shankari commented Jun 6, 2015

joshzarrabi commented Jun 6, 2015

AmplabJenkins commented Jun 10, 2015

AmplabJenkins commented Jun 10, 2015

AmplabJenkins commented Jun 10, 2015

AmplabJenkins commented Jun 11, 2015

AmplabJenkins commented Jun 11, 2015

AmplabJenkins commented Jun 16, 2015

AmplabJenkins commented Jun 16, 2015

AmplabJenkins commented Jun 16, 2015

AmplabJenkins commented Jun 18, 2015

AmplabJenkins commented Jun 18, 2015

AmplabJenkins commented Jun 18, 2015

shankari Jun 18, 2015

Choose a reason for hiding this comment

AmplabJenkins commented Jun 19, 2015

AmplabJenkins commented Jun 19, 2015

AmplabJenkins commented Jun 19, 2015

shankari Jun 19, 2015

Choose a reason for hiding this comment

shankari commented Jun 19, 2015

joshzarrabi commented Jun 19, 2015

AmplabJenkins commented Jun 24, 2015

AmplabJenkins commented Jun 24, 2015

AmplabJenkins commented Jun 24, 2015